Skip to content

Commit

Permalink
ci: single configuration for all OCaml packages
Browse files Browse the repository at this point in the history
I've run into trouble with multiple configurations in #143, since that
introduces dependencies between these local packages. This approach seems to
solve those problems, and I think it's somewhat more idiomatic anyway.
  • Loading branch information
samcowger committed Jan 22, 2025
1 parent cda048f commit 7bb2ab1
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 55 deletions.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: CN Server CI
name: OCaml CI

on:
push:
Expand Down Expand Up @@ -31,16 +31,16 @@ jobs:
with:
ocaml-compiler: ${{ matrix.ocaml-version }}

- name: Link lockfile
run: ln -s cn-lsp.opam.locked-${{ matrix.ocaml-version }} cn-lsp.opam.locked
- name: Link lockfiles
run: |
ln -s telemetry.opam.locked-${{ matrix.ocaml-version }} telemetry.opam.locked
ln -s cn-lsp.opam.locked-${{ matrix.ocaml-version }} cn-lsp.opam.locked
- name: Install server dependencies
run: opam install ./cn-lsp.opam --deps-only --locked -y
- name: Install dependencies
run: opam install . --deps-only --locked -y

- name: Build server
working-directory: ./cn-lsp
- name: Build
run: eval $(opam env) && dune build

- name: Run server tests
working-directory: ./cn-lsp
- name: Run tests
run: eval $(opam env) && dune test
46 changes: 0 additions & 46 deletions .github/workflows/telemetry-ci.yml

This file was deleted.

0 comments on commit 7bb2ab1

Please sign in to comment.