Jbuilder

[jaredly]

continuation of #30

[jaredly]

I think it leads to less confusion. Might has well have the directories have the same names as the compiled artifacts

[jaredly]

tbh I'd rather capitalize Internal_lib and all of the source files M1.re etc.

[jordwalke]

@jaredly If you prefer it, you can capitalize the files if you like - we could also setup a case sensitive travis build to let us know if that introduces any issues.

Regardless of if the package name is reason-native-project or reason-native-project, I think when it comes to actual code, the underscores are a little less familiar to JS developers. Is there a way to allow people to consume the result of this as ReasonNativeProject.InnerModules? Or is there some limitation in jBuilder?

[jaredly]

oh for sure! I was just converting from kebab to underscores, I'm totally fine with camel

[jordwalke]

Nice! I'll just wait until the CI passes then merge!

[jordwalke]

Happy to merge, but CI is not happy for some reason. Do you know what's up?

[jaredly]

oh funny, looks like there some duplication between travis.yml and test-with-version

[jordwalke]

Very good! Thank you @jaredly. Thank you @rgrinberg! This is a massive improvement.

[kennetpostigo]

Does the readme of the project need to be updated with the move to JBuilder?

[jordwalke]

yes! Do you feel like helping to document it?
( @jaredly might have suggestions for going beyond basic instructions but feel free to send PRs for ideas )