We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
derive
satisfies
I don't think that derive and satisfies should be declared in the global namespace.
equational_theories/equational_theories/MagmaLaw.lean
Line 46 in bfcc12a
Line 90 in bfcc12a
I suggest MagmaLaw.derive and MagmaLaw.satisfies as better names.
MagmaLaw.derive
MagmaLaw.satisfies
PS: I don't see a point of MagmaLaw being inside Law itself. However, it is another story.
MagmaLaw
Law
The text was updated successfully, but these errors were encountered:
Provided that the refactor can be done easily, I have no objection to this change.
Sorry, something went wrong.
No branches or pull requests
I don't think that
derive
andsatisfies
should be declared in the global namespace.equational_theories/equational_theories/MagmaLaw.lean
Line 46 in bfcc12a
equational_theories/equational_theories/MagmaLaw.lean
Line 90 in bfcc12a
I suggest
MagmaLaw.derive
andMagmaLaw.satisfies
as better names.PS: I don't see a point of
MagmaLaw
being insideLaw
itself. However, it is another story.The text was updated successfully, but these errors were encountered: