Skip to content

[Merged by Bors] - Chore: Rename files with Eisenstein series stuff.#14572

Closed
CBirkbeck wants to merge 1 commit intomasterfrom
eisenstein_Series_file_renaming
Closed

[Merged by Bors] - Chore: Rename files with Eisenstein series stuff.#14572
CBirkbeck wants to merge 1 commit intomasterfrom
eisenstein_Series_file_renaming

Conversation

@CBirkbeck
Copy link
Copy Markdown
Collaborator

@CBirkbeck CBirkbeck commented Jul 9, 2024

Accidentally created a Mathlib/NumberTheory/ModularForms/EisensteinSeries/ModularForm.lean file name. Made it a Basic.lean file and made the previous into a Defs.lean


Open in Gitpod

@CBirkbeck CBirkbeck requested a review from riccardobrasca July 9, 2024 15:59
@CBirkbeck CBirkbeck added the easy < 20s of review time. See the lifecycle page for guidelines. label Jul 9, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 9, 2024

PR summary c613c3afd3

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.NumberTheory.ModularForms.EisensteinSeries.ModularForm 1847 0 -1847 (-100.00%)
Mathlib.NumberTheory.ModularForms.EisensteinSeries.Basic 1396 1847 +451 (+32.31%)
Import changes for all files
Files Import difference
Mathlib.NumberTheory.ModularForms.EisensteinSeries.ModularForm -1847
Mathlib.NumberTheory.ModularForms.EisensteinSeries.Basic 451
Mathlib.NumberTheory.ModularForms.EisensteinSeries.Defs 1396

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/no_lost_declarations.sh short <optional_commit>

## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit>

@loefflerd
Copy link
Copy Markdown
Contributor

This seems to be purely a renaming PR, no new code added or removed, and I agree the new names are more sensible

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 9, 2024

🚀 Pull request has been placed on the maintainer queue by loefflerd.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jul 9, 2024
@loefflerd
Copy link
Copy Markdown
Contributor

(I am assuming that you meant to put an "awaiting-review" tag on this, since you requested a review from Riccardo.)

@riccardobrasca
Copy link
Copy Markdown
Member

(I am assuming that you meant to put an "awaiting-review" tag on this, since you requested a review from Riccardo.)

No more awaiting-review!

@riccardobrasca
Copy link
Copy Markdown
Member

Thanks!

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Jul 9, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jul 9, 2024
Accidentally created a `Mathlib/NumberTheory/ModularForms/EisensteinSeries/ModularForm.lean` file name. Made it a Basic.lean file and made the previous into a Defs.lean
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 9, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title Chore: Rename files with Eisenstein series stuff. [Merged by Bors] - Chore: Rename files with Eisenstein series stuff. Jul 9, 2024
@mathlib-bors mathlib-bors bot closed this Jul 9, 2024
@mathlib-bors mathlib-bors bot deleted the eisenstein_Series_file_renaming branch July 9, 2024 17:37
@adomani adomani mentioned this pull request Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

easy < 20s of review time. See the lifecycle page for guidelines. maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants