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] - refactor(logic/basic): classical -> root, root -> decidable #3812
Conversation
Could you add docs on the |
I thought about leaving the Most likely this PR has caused more lemmas to use classical logic that didn't intend to, since I didn't do a find/replace of root lemmas to |
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.
Otherwise LGTM.
bors merge |
This moves all logic lemmas with `decidable` instances into the `decidable` namespace, and moves or adds classical versions of these to the root namespace. This change hits a lot of files, mostly to remove the `classical.` prefix on explicit references to classical lemmas.
Pull request successfully merged into master. Build succeeded: |
This moves all logic lemmas with
decidable
instances into thedecidable
namespace, and moves or adds classical versions of these to the root namespace. This change hits a lot of files, mostly to remove theclassical.
prefix on explicit references to classical lemmas.