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

Logs UI #518

Merged
merged 29 commits into from
Jan 15, 2021
Merged

Logs UI #518

merged 29 commits into from
Jan 15, 2021

Conversation

firewall2142
Copy link
Contributor

@firewall2142 firewall2142 commented Jan 8, 2021

New:

  • Query results can be viewed on hover
  • Logs are now displayed in both vscode and emacs
  • Emacs: window layout can be easily changed
  • Emacs: Debug menu on the menu-bar
  • Vscode: debug menu in Run (in the activity bar)
  • LSP Server : added proof/debugFlags notification

@fblanqui
Copy link
Member

fblanqui commented Jan 9, 2021

Hi @ejgallego . Your review on this PR would be very much appreciated as it affects the LSP server.

@fblanqui fblanqui requested a review from ejgallego January 9, 2021 08:13
@@ -422,8 +485,8 @@ let main std log_file =
LIO.debug_fmt := F.formatter_of_out_channel debug_oc;

(* XXX: Capture better / per sentence. *)
let lp_oc = open_out "log-lp.txt" in
let lp_fmt = F.formatter_of_out_channel lp_oc in
(* let lp_oc = open_out "log-lp.txt" in *)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ejgallego What was log-lp.txt for as there is also /tmp/lambdapi_lsp_log.txt ?

@fblanqui fblanqui merged commit e4205c5 into Deducteam:master Jan 15, 2021
amelieled pushed a commit to amelieled/lambdapi that referenced this pull request Jan 24, 2021
* hover queries
* lsp logger
* turn off loggers before turning on
* vscode terminal
* emacs log buffer, customizable window layout
* update docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants