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

Fixed handling invalid tactic syntax #2

Closed
wants to merge 1 commit into from

Conversation

iforiq
Copy link

@iforiq iforiq commented Dec 10, 2024

Currently when a lean file has an ill-formed expression inside a proof tactic, an unhandled exception occurs. This change creates a test to isolate that, and then proposes a simple fix.

@lenianiva
Copy link
Owner

lenianiva commented Dec 10, 2024

This would not fix stanford-centaur/PyPantograph#44 plus it doesn't really provide information. A fix has been given here which is more general.

https://git.leni.sh/aniva/Pantograph/commit/37a5884be40790a946e0b8e26879ed3bbf509077

@iforiq
Copy link
Author

iforiq commented Dec 10, 2024

Makes sense

@iforiq iforiq closed this Dec 10, 2024
@iforiq iforiq deleted the b_fix_invalid_tactic branch December 10, 2024 06:39
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.

2 participants