Skip to content

Commit

Permalink
Remove ppx_expect since unavailable on 32-bit archs (3/n)
Browse files Browse the repository at this point in the history
- Remove --deps-only which is buggy. That, at least in opam 2.2.1, uses the dependencies from the repository rather than the local package when doing opam install . --deps-only
  • Loading branch information
jonahbeckford committed Sep 14, 2024
1 parent 63d6ba9 commit 33ba7ba
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/pages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
opam-pin: true
opam-depext: false # will complain that diskuvbox is missing
- name: Install Opam dependencies
run: opam install . --deps-only --with-doc
run: opam install . --with-doc

- name: Cache conda
uses: actions/cache@v3
Expand Down
2 changes: 1 addition & 1 deletion DEVELOPMENT.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ This is the Windows prerequisite for any development listed on this page:

```powershell
dkml init
opam install . --deps-only --with-test --yes
opam install . --with-test --yes
```

On Unix:
Expand Down
3 changes: 2 additions & 1 deletion ci/build-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -30,5 +30,6 @@ opamrun exec -- ocamlc -config
opamrun update

# Make your own build logic! It may look like ...
opamrun install . --deps-only --with-test --yes
# note: --deps-only is buggy. That, at least in opam 2.2.1, uses the dependencies from the repository rather than the local package when doing opam install . --deps-only
opamrun install . --with-test --yes
opamrun exec -- dune runtest

0 comments on commit 33ba7ba

Please sign in to comment.