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

[coq] Workspace handling code rework, part 2 #151

Merged
merged 1 commit into from
Jan 5, 2023

Conversation

ejgallego
Copy link
Owner

Some tweaks left out of #150 by mistake.

@ejgallego ejgallego added this to the 0.1.2 milestone Jan 5, 2023
@ejgallego ejgallego marked this pull request as draft January 5, 2023 10:37
@ejgallego ejgallego marked this pull request as ready for review January 5, 2023 11:13
Some tweaks left out of #150 by mistake.
@ejgallego ejgallego merged commit bf0f440 into main Jan 5, 2023
@ejgallego ejgallego deleted the tweaks_on_workspace_part_two branch January 5, 2023 11:37
ejgallego added a commit to ejgallego/opam-repository that referenced this pull request Jan 6, 2023
CHANGES:

------------------------

 - Send an error to the client if the client and server versions don't
   match (@ejgallego, ejgallego/coq-lsp#126)
 - Parse options -noinit, -indices-matter, and -impredicative-set from
   `_CoqProject` (@artagnon, @ejgallego, ejgallego/coq-lsp#140, ejgallego/coq-lsp#150)
 - Log file `log-lsp.txt` has been removed in favor of `coq-lsp.trace.server`
   (@artagnon, @ejgallego, ejgallego/coq-lsp#130, ejgallego/coq-lsp#148)
 - Added --bt flag to print a backtrace on error (@Alizter, ejgallego/coq-lsp#147)
 - A detailed view of Coq errors is now displayed in the info panel
   (@ejgallego, ejgallego/coq-lsp#128)
 - Coq "Notice" messages, such as the ones generated by `About` or
   `Search` are not shown anymore as diagnostics. Instead, they will
   be shown on the side panel when clicking on the corresponding
   command. The `show_notices_as_diagnostics` option allows to restore
   old behavior (@ejgallego, ejgallego/coq-lsp#128, fixes ejgallego/coq-lsp#125)
 - Print some more info about Coq workspace configuration (@ejgallego, ejgallego/coq-lsp#151)
 - Admit failed `Qed` by default; allow users to configure it
   (@ejgallego, ejgallego/coq-lsp#118, fixes ejgallego/coq-lsp#90)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

1 participant