add opam-check-npm-deps (#23)

* add opam-check-npm-deps * fix ci * downgrade version of react to match reason-react depexts
melange-re · Sep 8, 2023 · f5b9db7 · f5b9db7
1 parent f2dd8dc
commit f5b9db7
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 17 deletions.
diff --git a/Makefile b/Makefile
@@ -24,6 +24,7 @@ install: ## Install development dependencies
 	opam update
 	opam install -y . --deps-only
 	opam pin -y add $(project_name).dev .
+	opam exec opam-check-npm-deps
 
 .PHONY: build
 build: ## Build the project

diff --git a/melange-opam-template.opam b/melange-opam-template.opam
@@ -13,6 +13,7 @@ depends: [
   "melange" {>= "1.0.0"}
   "reactjs-jsx-ppx" {>= "1.0.0"}
   "reason-react" {>= "0.11.0"}
+  "opam-check-npm-deps" {dev} # todo: use with-dev-setup once 2.2 is out
   "ocaml-lsp-server" {dev}
   "dot-merlin-reader" {dev}
   "odoc" {with-doc}

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -4,8 +4,8 @@
     "serve": "webpack serve --open --mode development --entry ./_build/default/src/output/src/ReactApp.js"
   },
   "dependencies": {
-    "react": "^18.2.0",
-    "react-dom": "^18.2.0"
+    "react": "^17.0.0",
+    "react-dom": "^17.0.0"
   },
   "devDependencies": {
     "webpack": "^5.73.0",