Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: server -> client requests #3271

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

mhuisi
Copy link
Contributor

@mhuisi mhuisi commented Feb 7, 2024

This PR adds support for requests from the server to the client in the language server. It is based on #3014 and was developed during an experiment for #3247 that unfortunately did not go anywhere.

@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Feb 7, 2024
@leanprover-community-mathlib4-bot
Copy link
Collaborator

leanprover-community-mathlib4-bot commented Feb 7, 2024

Mathlib CI status (docs):

  • ❗ Std/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. (2024-02-07 10:29:57)
  • ❗ Std/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase ea665de4536061cb8b6a1b32f5b69f09d50ac335 --onto 8b8e001794e6a8b481d37f24fa77bb07797682a1. (2024-02-23 08:58:37)
  • ❗ Std/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 68eaf33e86689e3f303d38e2f071b5e8b7fd172f --onto 317adf42e92a4dbe07295bc1f0429b61bb8079fe. (2024-03-14 15:53:15)

@mhuisi mhuisi force-pushed the mhuisi/server-to-client-requests branch from cb0237f to b2826c7 Compare March 14, 2024 15:32
@mhuisi mhuisi marked this pull request as ready for review March 14, 2024 15:37
@mhuisi mhuisi added this pull request to the merge queue Mar 14, 2024
Merged via the queue into leanprover:master with commit 795e332 Mar 14, 2024
10 checks passed
github-merge-queue bot pushed a commit that referenced this pull request Mar 14, 2024
This PR fixes an issue where the file worker would not provide the
client with semantic tokens until the file had been elaborated
completely. The file worker now also tells the client to refresh its
semantic tokens after running "Restart File". This PR is based on #3271.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants