Skip to content

Commit 877d0e5

Browse files
fakenickelsMoOx
authored andcommitted
Make instructions clearer for git usage (#473)
* fix(bs-react-native): fix path name * fix monorepo install instructions and fix path script
1 parent 6006ad9 commit 877d0e5

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

‎README.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,9 @@ If you use this bindings from the git repo, be sure to use
2020
// ...
2121
"bs-dependencies": [
2222
// BsReactNative, current "stable" bindings
23-
"bs-react-native/bs-react-native",
23+
"bs-react-native-monorepo/bs-react-native",
2424
// ReactNative, WIP on zero-cost bindings (require bs-platform 5.0.0 & and reason-react hooks branch)
25-
"bs-react-native/reason-react-native"
25+
"bs-react-native-monorepo/reason-react-native"
2626
},
2727
// When used from git with the following path, bs-platform will use path that won't be working
2828
// (because deps are pointing to folder in folder)`
@@ -32,7 +32,7 @@ If you use this bindings from the git repo, be sure to use
3232
// the trick below will correct path in generated JavaScript files
3333
// and will just need to be removed when you won't use git directly anymore
3434
"js-post-build": {
35-
"cmd": "./node_modules/bs-react-native/git-monorepo-usage-trick"
35+
"cmd": "./node_modules/bs-react-native-monorepo/git-monorepo-usage-trick"
3636
}
3737
}
3838
```

‎git-monorepo-usage-trick‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@
77

88
# sed -i does not work in CI in a docker image on Linux, so let's use ex instead.
99
ex $1 <<EOF
10-
%s/require("bs-react-native\//require("bs-react-native\/bs-react-native\//g
11-
%s/from "bs-react-native\//from "bs-react-native\/bs-react-native\//g
12-
%s/require("reason-react-native\//require("bs-react-native\/reason-react-native\//g
13-
%s/from "reason-react-native\//from "bs-react-native\/reason-react-native\//g
10+
%s/require("bs-react-native\//require("bs-react-native-monorepo\/bs-react-native\//g
11+
%s/from "bs-react-native\//from "bs-react-native-monorepo\/bs-react-native\//g
12+
%s/require("reason-react-native\//require("bs-react-native-monorepo\/reason-react-native\//g
13+
%s/from "reason-react-native\//from "bs-react-native-monorepo\/reason-react-native\//g
1414
x
1515
EOF
1616

0 commit comments

Comments
 (0)