Skip to content

[Merged by Bors] - chore(Init/Data/Quot): better naming#16252

Closed
astrainfinita wants to merge 1 commit intomasterfrom
FR_quot_exact_name
Closed

[Merged by Bors] - chore(Init/Data/Quot): better naming#16252
astrainfinita wants to merge 1 commit intomasterfrom
FR_quot_exact_name

Conversation

@astrainfinita
Copy link
Copy Markdown
Collaborator

Quot.exactQuot.eqvGen_exact
Quot.EqvGen_soundQuot.eqvGen_sound


Open in Gitpod

@astrainfinita astrainfinita added the t-data Data (lists, quotients, numbers, etc) label Aug 29, 2024
@github-actions
Copy link
Copy Markdown

PR summary 208c7e2c36

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Init.Data.Quot 1 2 +1 (+100.00%)
Import changes for all files
Files Import difference
Mathlib.Init.Data.Quot 1

Declarations diff

+ Quot.eqvGen_exact
+ Quot.eqvGen_sound

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.

Copy link
Copy Markdown
Contributor

@Ruben-VandeVelde Ruben-VandeVelde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by Ruben-VandeVelde.

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Aug 31, 2024
@urkud
Copy link
Copy Markdown
Member

urkud commented Aug 31, 2024

Thanks! 🎉
bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Aug 31, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 31, 2024
`Quot.exact` ↦ `Quot.eqvGen_exact`
`Quot.EqvGen_sound` ↦ `Quot.eqvGen_sound`
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 31, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(Init/Data/Quot): better naming [Merged by Bors] - chore(Init/Data/Quot): better naming Aug 31, 2024
@mathlib-bors mathlib-bors bot closed this Aug 31, 2024
@mathlib-bors mathlib-bors bot deleted the FR_quot_exact_name branch August 31, 2024 19:37
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
`Quot.exact` ↦ `Quot.eqvGen_exact`
`Quot.EqvGen_sound` ↦ `Quot.eqvGen_sound`
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
`Quot.exact` ↦ `Quot.eqvGen_exact`
`Quot.EqvGen_sound` ↦ `Quot.eqvGen_sound`
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
`Quot.exact` ↦ `Quot.eqvGen_exact`
`Quot.EqvGen_sound` ↦ `Quot.eqvGen_sound`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors. t-data Data (lists, quotients, numbers, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants