[Merged by Bors] - chore: remove declarations deprecated between 2021-04-15 and 2025-10-15#38057
[Merged by Bors] - chore: remove declarations deprecated between 2021-04-15 and 2025-10-15#38057mathlib-nolints[bot] wants to merge 6 commits intomasterfrom
Conversation
PR summary 11873348d8Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
|
The commits deleting the empty sections were manual, right? We should look into running the global syntax linter on the output of this workflow and act on its findings. |
|
Yes, they were manual. |
|
The script git diff upstream/master...deprecated-decls | grep -5 "^ --[^-]"finds comments that are present in For instance, running it on the commit where @Parcly-Taxel identified the extra comment that could be removed: git diff upstream/master...85044fb2bff2101617e4055cc4f92e226ea2c72d | grep -5 "^ --[^-]"identifies the comment that was flagged. With a bit more effort, the command could also show just the hits where the comment precedes a deleted line, but since there are so few false positives, maybe it is not worth fixing. Anyway, I am mentioning this, since the PR description might suggest it (below the fold) as a test to perform. |
|
#38124 is running the generic syntax linter on this PR. |
|
I removed the pairs Everything looks good to me! bors d+ |
|
✌️ mathlib-nolints[bot] can now approve this pull request. To approve and merge a pull request, simply reply with |
|
bors d=bryangingechen |
|
✌️ bryangingechen can now approve this pull request. To approve and merge a pull request, simply reply with |
|
bors r+ |
…15 (#38057) I am happy to remove some deprecated declarations for you! Please check if there are any remaining stray comments or other issues before merging. Co-authored-by: Bryan Gin-ge Chen <bryangingechen@gmail.com> Co-authored-by: Damiano Testa <adomani@gmail.com>
|
I couldn't find the PR for the global syntax linter, but maybe we should add it as an extra step to the deprecated declarations workflow in that PR. |
|
Pull request successfully merged into master. Build succeeded: |
I am happy to remove some deprecated declarations for you!
Please check if there are any remaining stray comments or other issues before merging.
workflow run for this PR