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

Fix coq analyze text logic #1597

Closed
wants to merge 1 commit into from
Closed

Fix coq analyze text logic #1597

wants to merge 1 commit into from

Conversation

gandarez
Copy link
Contributor

@gandarez gandarez commented Nov 9, 2020

the current Coq analysis the actual value for completing a proof is Qed but also qed can appear on it. So I applied a regex to validate it.

https://coq.inria.fr/refman/proof-engine/proof-handling.html?highlight=qed#coq:cmd.qed
https://coq.inria.fr/refman/proof-engine/tactics.html#coq:tacn.tauto

@gandarez
Copy link
Contributor Author

@Anteru any update here?

@birkenfeld birkenfeld closed this Feb 12, 2021
@birkenfeld
Copy link
Member

Seems like this has been superseded by #1648.

@gandarez gandarez deleted the bugfix/coq branch February 12, 2021 20:27
@Anteru Anteru self-assigned this Feb 14, 2021
@Anteru Anteru added the changelog-update Items which need to get mentioned in the changelog label Feb 14, 2021
@Anteru Anteru added this to the 2.8 milestone Feb 14, 2021
@Anteru Anteru removed the changelog-update Items which need to get mentioned in the changelog label Feb 14, 2021
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.

None yet

3 participants