make Js.Json.t use Boolean(bool) instead of explicit true/false (#6421)

* make Js.Json.t use Boolean(bool) instead of explicit true/false * changelog * move changelog entry to breaking changes
rescript-lang · Oct 4, 2023 · 167c586 · 167c586
1 parent b8c4157
commit 167c586
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,6 +29,7 @@
 
 #### :boom: Breaking Change
 - Add smart printer for pipe-chains.  https://github.com/rescript-lang/rescript-compiler/pull/6411 (the formatter will reformat existing code in certain cases)
+- `Js.Json.t` now uses `Boolean(bool)` instead of explicit `@as(true) True | @as(false) False`. https://github.com/rescript-lang/rescript-compiler/pull/6421
 
 #### :nail_care: Polish
 

diff --git a/jscomp/others/js_json.res b/jscomp/others/js_json.res
@@ -26,8 +26,7 @@
 
 @unboxed
 type rec t =
-  | @as(false) False
-  | @as(true) True
+  | Boolean(bool)
   | @as(null) Null
   | String(string)
   | Number(float)

diff --git a/jscomp/others/js_json.resi b/jscomp/others/js_json.resi
@@ -32,8 +32,7 @@
 
 @unboxed /** The JSON data structure */
 type rec t =
-  | @as(false) False
-  | @as(true) True
+  | Boolean(bool)
   | @as(null) Null
   | String(string)
   | Number(float)