Documentation update (mentions Melange and fixes a few things) #765
Conversation
… Improve-documentation-and-add-melange * 'source' of github.com:/reasonml/reasonml.github.io: Replace yarn with npm on README Use npm instead of yarn Proofread README by andyg0808
|
|
||
| > Melange strives to provide the best integration with both the OCaml and JavaScript ecosystems | ||
|
|
||
| If you have experience with JavaScript and the ecosystem we recommend Melange, over other alternatives such as [Js_of_ocaml](compiling-to-js-with-jsoo.md). |
There was a problem hiding this comment.
This looks broken: https://reasonml-github-io-reasonml.vercel.app/docs/en/compiling-to-js-with-jsoo
is it intended to be outside of the docs list?
There was a problem hiding this comment.
I believe we should add in on the sidebar (which will fix this issue) but since it's a little rough page I'm unsure
There was a problem hiding this comment.
Removed the link, when this page is ready can be pointed out in the sidebar and in the content
| @@ -2,73 +2,102 @@ | |||
| title: Installation | |||
There was a problem hiding this comment.
I see this section as "Native installation" and the one in the Melange section as "Js installation". Not sure if it'd be worth to separate these 2 cases more clearly (prob to do somewhere in the future).
There was a problem hiding this comment.
The biggest difference is just melange right?
We might need to revisit this in the future, indeed.
Co-authored-by: José Nogueira <ze@thatportugueseguy.com>
Co-authored-by: Javier Chávarri <javier.chavarri@gmail.com>
This PR updates the Reason documentation with a few aditions and a few changes:
installation-esyand removes it from the sidebarinstallationDeployed here: https://reasonml-github-io-reasonml.vercel.app/docs/en/what-and-why
reasonneeds to be added a dependency #619npm installto quickstart script #713