Skip to content

Commit

Permalink
Fix CI so that it runs properly on Windows (#7)
Browse files Browse the repository at this point in the history
  • Loading branch information
Niols authored May 9, 2023
1 parent c0b5a39 commit c6c693d
Showing 1 changed file with 3 additions and 15 deletions.
18 changes: 3 additions & 15 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -77,10 +77,8 @@ jobs:
opam-depext: true ## this is the default; set here explicitly

- name: Install Topiary
shell: bash
run: |
## Install Topiary then report where it is and in which version.
set -euC
echo '::group::Install Topiary'
opam install topiary
echo '::endgroup::'
Expand All @@ -92,33 +90,23 @@ jobs:
echo '::endgroup::'
- name: Run a smoke test
shell: bash
run: |
## Run a tiny smoke test; checking that Topiary manages to start,
## parse a simple OCaml command and output the right thing.
set -euC
echo 'open Foo' > expected.ml
echo 'open Foo' \
| opam exec -- topiary --language ocaml \
> result.ml
echo 'open Foo' | opam exec -- topiary --language ocaml > result.ml
diff expected.ml result.ml
## NOTE: Proper tests are disabled because they actually do not pass for
## Topiary v0.1.0. These should be re-enabled in future versions.

# - name: Run tests
# shell: bash
# run: |
# set -euC
# ## Test on OCaml implementation files (.ml)
# cat topiary/tests/samples/input/ocaml.ml \
# | opam exec -- topiary --language ocaml-implementation \
# > ocaml.ml
# cat topiary/tests/samples/input/ocaml.ml | opam exec -- topiary --language ocaml-implementation > ocaml.ml
# diff topiary/tests/samples/expected/ocaml.ml ocaml.ml
# ## Test on OCaml interface files (.mli)
# cat topiary/tests/samples/input/ocaml.mli \
# | opam exec -- topiary --language ocaml-interface \
# > ocaml.mli
# cat topiary/tests/samples/input/ocaml.mli | opam exec -- topiary --language ocaml-interface > ocaml.mli
# diff topiary/tests/samples/expected/ocaml.mli ocaml.mli

docker-based:
Expand Down

0 comments on commit c6c693d

Please sign in to comment.