style(Order): fix whitespace#30690
style(Order): fix whitespace#30690grunweg wants to merge 1 commit intoleanprover-community:masterfrom
Conversation
Extracted from leanprover-community#30658. Found by extending the commandStart linter to proof bodies.
PR summary 7c64d37d34Import 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
|
grunweg
left a comment
There was a problem hiding this comment.
Thanks for doing this work, Damiano. I like all changes which are not related to removing a space before not, but am on the fence about most of those. Let me split this PR in two, and bors r+ the "good" half.
|
#30762 is the good subset, which I've just borsed |
|
I agree that for the spacing after not, there is no clear consensus on what is the preferred form. I'd like it if we picked one style and applied it consistently. But maybe we should ask others what they think. |
|
This pull request has conflicts, please merge |
|
The remaining changes in this PR are undesirable: closing. |
Extracted from #30658. Found by extending the commandStart linter to proof bodies.