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
[Merged by Bors] - fix(meta/expr): adjust is_likely_generated_binder_name to lean#490 #4915
Conversation
Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts `name.is_likely_generated_binder_name`, which checks whether a binder name was likely generated by Lean (rather than given by the user).
@gebner could you take a look at this? Particularly whether the docs are still accurate. |
Do you mind adding a test so that we'll remember this if we change the name again in the future? |
I agree with Bryan, this deserves a test. bors d+ |
✌️ JLimperg can now approve this pull request. To approve and merge a pull request, simply reply with |
Added the test case. Thanks for the suggestion and reviews everyone! bors r+ |
…4915) Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts `name.is_likely_generated_binder_name`, which checks whether a binder name was likely generated by Lean (rather than given by the user).
Build failed: |
The build failure seems to be caused by the problem discussed in this Zulip thread. Retrying... bors r+ |
…4915) Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts `name.is_likely_generated_binder_name`, which checks whether a binder name was likely generated by Lean (rather than given by the user).
Build failed (retrying...): |
…4915) Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts `name.is_likely_generated_binder_name`, which checks whether a binder name was likely generated by Lean (rather than given by the user).
Build failed: |
bors r+ |
…4915) Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts `name.is_likely_generated_binder_name`, which checks whether a binder name was likely generated by Lean (rather than given by the user).
Pull request successfully merged into master. Build succeeded: |
…4915) Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts `name.is_likely_generated_binder_name`, which checks whether a binder name was likely generated by Lean (rather than given by the user).
Lean PR 490 changed Lean's strategy for generating binder names. This PR adapts
name.is_likely_generated_binder_name
, which checks whether a binder name waslikely generated by Lean (rather than given by the user).