-
Notifications
You must be signed in to change notification settings - Fork 251
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] - feat: port Order.Zorn
#1254
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
bors d+
✌️ ADedecker can now approve this pull request. To approve and merge a pull request, simply reply with |
bors r+ |
Co-authored-by: Heather Macbeth <25316162+hrmacbeth@users.noreply.github.com>
Build failed:
|
It failed to build, apparently complaining about the local notation already being used in another file:
Strange because this file successfully built before. I don't know if the problem was in deleting the comment -- mathport name: «expr ≺ » or in merging master. I'll try merging master first. |
I've encountered this before, this is a bug (see here). The temporary fix is to give a name to the notation. I'm not at my computer right now, but you can have a look at what I did in |
bors r+ |
Co-authored-by: Heather Macbeth <25316162+hrmacbeth@users.noreply.github.com>
Pull request successfully merged into master. Build succeeded! And happy new year! 🎉 |
Order.Zorn
Order.Zorn
No description provided.