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

Formatting Proof view #749

Closed
oddcoder opened this issue Mar 6, 2024 · 4 comments · Fixed by #773
Closed

Formatting Proof view #749

oddcoder opened this issue Mar 6, 2024 · 4 comments · Fixed by #773
Labels
bug Something isn't working

Comments

@oddcoder
Copy link

oddcoder commented Mar 6, 2024

Legacy vscoq allows proofs to be formatted nicely, But this is no the case for the current vscoq as shown below:

image

This is makes it extremely hard to follow computer science related proofs as the induction principles generated are very large and wordy. I often find myself having to copy paste it to a new tab and reformatting it by hand just to understand what I am trying to do.

@rtetley
Copy link
Collaborator

rtetley commented Mar 12, 2024

Hi and thanks for reporting ! Coud you give me an example to work with ? As far as I remember we had fixed the formatting issues when we switched to using ppstrings.

@rtetley rtetley added the bug Something isn't working label Mar 12, 2024
@Lee-Janggun
Copy link

https://gitlab.mpi-sws.org/iris/examples should work. Before, both the coq context and the proof goals were formatted nicely with good indentations. However, now there are no indentations.

@gares
Copy link
Collaborator

gares commented Mar 20, 2024

Hi and thanks for reporting ! Coud you give me an example to work with ? As far as I remember we had fixed the formatting issues when we switched to using ppstrings.

I can reproduce the same bug without Iris, it is sufficient to have a large goal.
I believe this is a regression, it seems formatting (like applying the box language) is not performed anymore.

@gares
Copy link
Collaborator

gares commented Mar 22, 2024

Bug:
Screenshot from 2024-03-22 10-33-51
Expected:
Screenshot from 2024-03-22 10-34-23
Documentation of the box module Coq uses behind the scenes:
https://v2.ocaml.org/api/Format.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants