-
Notifications
You must be signed in to change notification settings - Fork 342
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(Nat): m / n = 0 ↔ n = 0 ∨ m < n
#19505
base: master
Are you sure you want to change the base?
Conversation
From LeanCamCombi
PR summary 04f3842e4cImport changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diffNo declarations were harmed in the making of this PR! 🐙 You can run this locally as follows## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/declarations_diff.sh long <optional_commit> The doc-module for No changes to technical debt.You can run this locally as
|
Could you add "Moves: ..." to the PR description, as you can't deprecate the old names? Another option which I recall being taken once is to keep the old lemma's name, deprecate it, and use |
Confusingly (cc @eric-wieser), "Moves" means "renames", not "statement change" nor "relocation". So this PR contains no "Move". |
Perhaps I read this PR wrong, but if I have
and change it to
then this should be recorded as a move from |
Then do you want me to add
to the PR description? A less silly way to look at things is that I strengthened three lemmas. |
From LeanCamCombi