You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 6, 2018. It is now read-only.
ReasonML's editor support has shifted towards a language-server which itself runs bsb whenever needed, automatically regenerating the js files and piping errors back to the specific editor (I use VScode and it does work this way). The two bsb's can apparently conflict on such things as the single .merlin file.
Would you be open to separating the bsb -watch? I would suggest it not be included, and the README steps can tell the user that bsb-(re)compiling is needed e.g. through their editor setup or by them manually keeping a bsb -watch running in terminal (ReasonML and Bucklescript users probably know about bsb -make-world -clean-world -w already).
The text was updated successfully, but these errors were encountered:
It looks like calling
bsb -watch
and the native packager go together here: https://github.com/react-community/reason-react-native-scripts/blob/master/src/util/packager.js#L72-L75ReasonML's editor support has shifted towards a language-server which itself runs
bsb
whenever needed, automatically regenerating the js files and piping errors back to the specific editor (I use VScode and it does work this way). The two bsb's can apparently conflict on such things as the single.merlin
file.Would you be open to separating the
bsb -watch
? I would suggest it not be included, and the README steps can tell the user that bsb-(re)compiling is needed e.g. through their editor setup or by them manually keeping absb -watch
running in terminal (ReasonML and Bucklescript users probably know aboutbsb -make-world -clean-world -w
already).The text was updated successfully, but these errors were encountered: