Skip to content

Commit

Permalink
Merge pull request #879 from ejgallego/bump_coq
Browse files Browse the repository at this point in the history
[vendor] Bump Coq
  • Loading branch information
ejgallego authored Dec 6, 2024
2 parents 08f4606 + 61052e8 commit 3fa7fa1
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ jobs:
run: |
opam install lwt logs # Also build pet-server
opam install memprof-limits # We need to do this to avoid coq-lsp rebuilding Coq below due to deptops
opam install vendor/coq/{coq-core,rocq-core,coq-stdlib,coqide-server,coq}.opam
opam install vendor/coq/{coq-core,rocq-core,stdlib/coq-stdlib,coqide-server,coq}.opam
- name: Install `coq-lsp` into OPAM switch
run: opam install .
Expand Down
13 changes: 7 additions & 6 deletions test/server/src/Check.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,19 +31,20 @@ test("Open file with wrong URI", async () => {
"Definition a := 3.",
);

await languageServer.sendNotification(
Protocol.DidOpenTextDocumentNotification.type,
{
textDocument,
},
);
let p = new Promise<Protocol.PublishDiagnosticsParams>((resolve) => {
languageServer.onNotification(
Protocol.PublishDiagnosticsNotification.type,
resolve,
);
});

await languageServer.sendNotification(
Protocol.DidOpenTextDocumentNotification.type,
{
textDocument,
},
);

const checkDiags = async (params: Protocol.PublishDiagnosticsParams) => {
if (params.diagnostics.length == 0) return "ok";
else throw "wrong number of diags";
Expand Down
2 changes: 1 addition & 1 deletion vendor/coq
Submodule coq updated 1541 files

0 comments on commit 3fa7fa1

Please sign in to comment.