Update contributing and ppx

rescript-labs · Apr 26, 2024 · 093a4f9 · 093a4f9
1 parent 2ddcedc
commit 093a4f9
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.github/workflows/ppx-pr.yaml b/.github/workflows/ppx-pr.yaml
@@ -14,8 +14,14 @@ jobs:
       with:
         ocaml-version: 4.12.1  # replace with your desired version
     - name: Install Dune
-      run: opam install dune
-    - name: Install Dependencies
+      run: |
+        opam install dune
+        eval $(opam env)
+    - name: Install OCaml Dependencies
+      run: |
+        cd ppx_src
+        opam install -y . --deps-only
+    - name: Install Yarn Dependencies
       run: yarn
     - name: Build PPX
       run: yarn build-ppx

diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -33,7 +33,7 @@ You would need to have [opam](https://opam.ocaml.org) installed.
 - cd ppx_src;
 - opam switch create . 4.12.1 --deps-only # If it's the first time you create a switch it can take a while.
 - eval $(opam env)
-- opam install -y ocaml-lsp-server dune ocaml ppx_tools_versioned # Install ppx's dependencies inside the switch
+- opam install -y . --deps-only
 - Profit, this should make your editor a little more smart
 
 ### Testing