Skip to content

[Merged by Bors] - chore: remove now-superfluous disabling of the header linter#24028

Closed
grunweg wants to merge 2 commits intomasterfrom
MR-cleanup-deprecations
Closed

[Merged by Bors] - chore: remove now-superfluous disabling of the header linter#24028
grunweg wants to merge 2 commits intomasterfrom
MR-cleanup-deprecations

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Apr 14, 2025

PR #24027 removes the need to disable the header linter before module
deprecations.


Open in Gitpod

@grunweg grunweg added the easy < 20s of review time. See the lifecycle page for guidelines. label Apr 14, 2025
@github-actions
Copy link
Copy Markdown

PR summary e40dd0305f

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No 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 script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@adomani
Copy link
Copy Markdown
Contributor

adomani commented Apr 14, 2025

Thanks!

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 14, 2025

✌️ grunweg can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@ghost ghost added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Apr 14, 2025
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

@adomani
Copy link
Copy Markdown
Contributor

adomani commented Apr 14, 2025

It turns out that I intended the syntax to be as close as possible to deprecated, but I forgot a :=: would you mind if I hijack this PR to also address the := issue?

@adomani
Copy link
Copy Markdown
Contributor

adomani commented Apr 14, 2025

Michael, if you want to merge #24032 into this PR and close the other one, feel free to do so!

@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Apr 14, 2025

Sure, feel free to merge that PR into this one.

@grunweg
Copy link
Copy Markdown
Contributor Author

grunweg commented Apr 14, 2025

Actually: given the other CI is still running, let's just merge this PR. #20979 is another test case for #24032 anyway.
Thanks for the quick review!
bors r+

mathlib-bors bot pushed a commit that referenced this pull request Apr 14, 2025
PR #24027 removes the need to disable the header linter before module
deprecations.



Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 14, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: remove now-superfluous disabling of the header linter [Merged by Bors] - chore: remove now-superfluous disabling of the header linter Apr 14, 2025
@mathlib-bors mathlib-bors bot closed this Apr 14, 2025
@mathlib-bors mathlib-bors bot deleted the MR-cleanup-deprecations branch April 14, 2025 11:14
tannerduve pushed a commit that referenced this pull request May 13, 2025
PR #24027 removes the need to disable the header linter before module
deprecations.



Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). easy < 20s of review time. See the lifecycle page for guidelines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants