Skip to content

[Merged by Bors] - chore(RepresentationTheory/GroupCohomology): add deprecation module#25878

Closed
101damnations wants to merge 243 commits intoleanprover-community:masterfrom
101damnations:gphombasicmovedfiles
Closed

[Merged by Bors] - chore(RepresentationTheory/GroupCohomology): add deprecation module#25878
101damnations wants to merge 243 commits intoleanprover-community:masterfrom
101damnations:gphombasicmovedfiles

Conversation

@101damnations
Copy link
Copy Markdown
Collaborator

@101damnations 101damnations commented Jun 14, 2025

@101damnations 101damnations changed the title chore(RepresentationTheory/GroupCohomology); chore(RepresentationTheory/GroupCohomology): add deprecation module Jun 14, 2025
@github-actions github-actions bot added the t-algebra Algebra (groups, rings, fields, etc) label Jun 14, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 14, 2025

PR summary 260a59832c

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.RepresentationTheory.GroupCohomology.Resolution (new file) 1716
Mathlib.RepresentationTheory.GroupCohomology.Basic (new file) 1717
Mathlib.RepresentationTheory.GroupCohomology.LowDegree (new file) 1915
Mathlib.RepresentationTheory.GroupCohomology.Functoriality (new file) 1916
Mathlib.RepresentationTheory.GroupCohomology.Hilbert90 (new file) 1920

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).

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jun 14, 2025
@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Jun 14, 2025

I can comment on the "how to deprecate" aspect:
if #25868 just moves files, then the new file should import just the new module. If a file got split, importing the old imports is probably appropriate. The idea is that an import of a deprecated_module should be replace by the imports in the file declaring it, so the import should be set up so this just works. If I imported module A which was renamed to B, I also want to import the declarations within that module --- so should import A.

It looks like this is the case here - so can you adjust the last commit, please?

@grunweg grunweg added the awaiting-author A reviewer has asked the author a question or requested changes. label Jun 14, 2025
@101damnations
Copy link
Copy Markdown
Collaborator Author

Thanks very much! I've tried to apply this.

@101damnations 101damnations removed the awaiting-author A reviewer has asked the author a question or requested changes. label Jun 14, 2025
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jun 16, 2025
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

@jcommelin
Copy link
Copy Markdown
Member

Thanks 🎉

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Jun 16, 2025
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 16, 2025

Canceled.

@jcommelin
Copy link
Copy Markdown
Member

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Jun 17, 2025
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 17, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(RepresentationTheory/GroupCohomology): add deprecation module [Merged by Bors] - chore(RepresentationTheory/GroupCohomology): add deprecation module Jun 17, 2025
@mathlib-bors mathlib-bors bot closed this Jun 17, 2025
Rida-Hamadani pushed a commit to Rida-Hamadani/mathlib4 that referenced this pull request Jun 24, 2025
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants