Playground New ReScript JS API 2020

.circleci/config.yml

@@ -10,6 +10,7 @@ jobs: # a collection of steps
           name: "Setting env variables"
           command: |
             echo 'export BS_TRAVIS_CI=1' >> $BASH_ENV
+            echo 'export NINJA_FORCE_REBUILD=1' >> $BASH_ENV
             echo 'export OCAMLRUNPARAM="b"' >> $BASH_ENV
       - run:    
           name: "Check env variables"

.github/ISSUE_TEMPLATE.md

@@ -1,5 +1,5 @@
 Thank you for filing! Check list:
-- [ ] Is it a bug? Usage questions should often be asked in the [forum](https://reasonml.chat) instead.
+- [ ] Is it a bug? Usage questions should often be asked in the [forum](https://forum.rescript-lang.org) instead.
 - [ ] Concise, focused, friendly issue title & description.
 - [ ] A [minimal, reproducible example](http://sscce.org).
   - [ ] Ideally in OCaml syntax for now. To isolate the problem caused by either Reason or the New Syntax.

.gitmodules

@@ -1,10 +1,12 @@
 [submodule "ocaml"]
 	path = ocaml
 	url = https://github.com/rescript-lang/ocaml
+	ignore = untracked
 [submodule "syntax"]
 	path = syntax
 	url = https://github.com/rescript-lang/syntax
 	branch = master
 [submodule "ninja"]
 	path = ninja
-	url = git@github.com:rescript-lang/ninja.git
+	url = https://github.com/rescript-lang/ninja
+	ignore = untracked

.npmignore

@@ -38,4 +38,6 @@ package/
 .nyc_output/
 themes
 vendor/js_of_ocaml.bc
-syntax/
+syntax/
+*.log
+ninja

CONTRIBUTING.md

@@ -50,6 +50,7 @@ If this doesn't work (rare), then:
 If you'd like to use your modified ReScript like an end-user, try:
 
 ```sh
+npm uninstall -g bs-platform # a cache-busting uninstall is needed, but only for npm >=7
 BS_TRAVIS_CI=1 npm install -g .
 ```
 

Changes.md

@@ -1,5 +1,127 @@
 `*` means  potential break changes
 
+# 8.4.1
+
+- Syntax submodule upgrades from 7f5c968 to 7cc70c9
+- #4856 #4858
+  Improve code generation for pattern match:
+  Input:
+  ```res
+  type t =
+    | A 
+    | B
+    | C 
+    | D (int )
+    | E (int)
+
+  let f = x => {
+    switch x {
+        | A => 0
+        | B => 1
+        | C => 2
+        | D (x) => x 
+        | E (x) => x + 1
+    }
+  }    
+  ```
+  Output was:
+  ```js
+    function f(x) {
+      if (typeof x !== "number") {
+        if (x.TAG) {
+        return x._0 + 1 | 0;
+      } else {
+        return x._0;
+      }
+
+      switch (x) {
+        case /* A */0 :
+          return 0;
+        case /* B */1 :
+          return 1;
+        case /* C */2 :
+          return 2;      
+    }
+  }
+  ```
+
+  Now:
+  ```js
+  function f(x) {
+    if (typeof x !== "number") {
+      if (x.TAG === /* D */0) {
+        return x._0;
+      } else {
+        return x._0 + 1 | 0;
+      }
+    }
+    switch (x) {
+      case /* A */0 :
+          return 0;
+      case /* B */1 :
+          return 1;
+      case /* C */2 :
+          return 2;
+
+    }
+  }
+  ```
+
+
+
+- #4855 *internal changes*
+  changes to compiler-libs will trigger a rebuild of the compiler, this allows us to
+  see how changes of compiler-libs affect bsc.exe quickly
+
+- #4850 replace ocp-ocamlres with a lightweight nodejs script, get rid of such dev dependency
+
+- #4854 #4848 #4847 #4844 #4836 #4826 #4824
+
+  Pinned packages support and `-make-world` respect changes of dependencies
+
+- #4840 #4841 more robust handling of removing stale output
+
+
+- #4831 use relative paths in the command line
+  It will be expanded to absolute path right after the compiler see the path,
+  such changes work better with the underlying ninja build engine, and should perform slightly better
+
+- #4828 no need pass -o for compiling, inferred directly (with namespace support too)
+
+- #4827 *internal* the dev version of bsc now behave roughly the same as the released version
+
+- #4825 fix a typo in the warning `%@string` -> `@string`
+
+- #4823 introduce a new warning 109: toplevel expression is expected to have type unit
+  It is turned on as warn-error by default. This warning is introduced to avoid partial application errors in a curried language
+
+- #4822 more robust hanlding of : ignore warnings and warn-error when bsb is building dependencies
+
+
+
+# 8.3.3
+This is a bug release for 8.3.*
+- #4817 *internal* add an option RES_SKIP_STDLIB_CHECK so that 
+  for a true monorepo, it does not need follow `node_modules` layout
+- #4807 #4815 remove unused code in refmt parser *a lot* (around 50_000 loc)
+  on darwin, the binary size is dropped fom 9.69M to 8.48M
+- #4808 add back basic-reason theme to avoid breakage for existing docs  
+- #4806 Fix broken ocaml build with gcc 10
+- #4804 restore back-wards compatibility with `build statement` in generated ninja files
+- #4803 fix the bsb build schema location in the error message
+- #4802 proper error message when bsconfig.json is missing
+- #4801 add a sanity check for name field in bsconfig.json to match real package name
+- #4810 #4784 regressions for weird indentation in warning output
+
+
+# 8.3.1
+This is a minor bug fix release for 8.3.0
+- capture warnings when rebuild without enforce warn-as-error
+- #4716 internal, make ninja a submodule in dev process
+- #4722 better dataflow for cases like `let {a;b} as obj = ...`
+- no need call `caml_enter_blocking_section` for single threaded compiler
+- #4739 fix the interaction of exotic filenames like `[id]` with the build system.
+
 # 8.3
 
 - #4694, #4712 improving/customizing the underlying ninja build system, better performance  

docs/docson/build-schema.json

@@ -404,6 +404,10 @@
             "$ref": "#/definitions/dependencies",
             "description": "OCaml/Reason dev dependencies of the library, like in package.json. Currently searches in `node_modules`"
         },
+        "pinned-dependencies" : {
+            "$ref": "#/definitions/dependencies",
+            "description": "Those dependencies are pinned (since version 8.4)"
+        },
         "generators": {
             "type": "array",
             "items": {

jscomp/artifacts.json

@@ -198,7 +198,6 @@
     "map.js",
     "marshal.js",
     "moreLabels.js",
-    "nativeint.js",
     "node_buffer.js",
     "node_child_process.js",
     "node_fs.js",
@@ -364,7 +363,6 @@
     "map.js",
     "marshal.js",
     "moreLabels.js",
-    "nativeint.js",
     "node_buffer.js",
     "node_child_process.js",
     "node_fs.js",
@@ -488,7 +486,6 @@
     "map.cmi",
     "marshal.cmi",
     "moreLabels.cmi",
-    "nativeint.cmi",
     "node_buffer.cmi",
     "node_child_process.cmi",
     "node_fs.cmi",
@@ -619,7 +616,6 @@
     "map.cmt",
     "marshal.cmt",
     "moreLabels.cmt",
-    "nativeint.cmt",
     "node_buffer.cmt",
     "node_child_process.cmt",
     "node_fs.cmt",
@@ -724,7 +720,6 @@
     "map.cmti",
     "marshal.cmti",
     "moreLabels.cmti",
-    "nativeint.cmti",
     "node_process.cmti",
     "obj.cmti",
     "parsing.cmti",
@@ -857,7 +852,6 @@
     "mapm.cppo.ml",
     "marshal.ml",
     "moreLabels.ml",
-    "nativeint.ml",
     "node_buffer.ml",
     "node_child_process.ml",
     "node_fs.ml",
@@ -969,7 +963,6 @@
     "mapm.cppo.mli",
     "marshal.mli",
     "moreLabels.mli",
-    "nativeint.mli",
     "node_process.mli",
     "obj.mli",
     "parsing.mli",
@@ -1014,6 +1007,7 @@
     "littlecase.js",
     "ninja.js",
     "ninjaFactory.js",
+    "pack.js",
     "prebuilt.js",
     "prepublish.js",
     "release.js",