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

feat(specialize): additional goals come after the main goal #777

Closed
wants to merge 1 commit into from

Conversation

kim-em
Copy link
Contributor

@kim-em kim-em commented Oct 28, 2022

This reverts #530.

Lean 4's specialize puts the additional goals after the main goal. Rather than writing an additional mini-tactic to preserve Lean 3 behaviour, I propose we just backport to achieve this behaviour. It's a relatively minor change to mathlib to adapot.

I couldn't find an account of why #530 wanted to change the order in the first place.

@kim-em
Copy link
Contributor Author

kim-em commented Oct 28, 2022

The alternative, I guess would be to see if we can change the Lean 4 behaviour.

The required diff in mathlib is at https://github.com/leanprover-community/mathlib/compare/revert_530.

@gebner
Copy link
Member

gebner commented Nov 1, 2022

@kim-em
Copy link
Contributor Author

kim-em commented Nov 5, 2022

The Lean 4 behaviour was changed to match Lean 3 in leanprover/lean4#1796.

@kim-em kim-em closed this Nov 5, 2022
@kim-em kim-em deleted the revert_530 branch November 5, 2022 12:40
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