Skip to content

[Merged by Bors] - refactor(SetTheory/Ordinal/Principal): fix bad names#17842

Closed
vihdzp wants to merge 1 commit intomasterfrom
vi.princ_renames
Closed

[Merged by Bors] - refactor(SetTheory/Ordinal/Principal): fix bad names#17842
vihdzp wants to merge 1 commit intomasterfrom
vi.princ_renames

Conversation

@vihdzp
Copy link
Copy Markdown
Collaborator

@vihdzp vihdzp commented Oct 16, 2024

This PR performs the following renames:

  • principal_add_isLimitisLimit_of_principal_add
  • principal_mul_isLimitisLimit_of_principal_mul
  • opow_principal_add_of_principal_addprincipal_add_opow_of_principal_add
  • mul_principal_add_is_principal_addprincipal_add_mul_of_principal_add

Open in Gitpod

@github-actions
Copy link
Copy Markdown

PR summary 0e8b453d08

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ isLimit_of_principal_add
+ isLimit_of_principal_mul
+ principal_add_mul_of_principal_add
+ principal_add_opow_of_principal_add

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
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

bors d+

one optional (family of) suggestions

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 1, 2024

✌️ vihdzp 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 Nov 1, 2024
@vihdzp
Copy link
Copy Markdown
Collaborator Author

vihdzp commented Nov 1, 2024

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Nov 1, 2024
This PR performs the following renames:

- `principal_add_isLimit` → `isLimit_of_principal_add`
- `principal_mul_isLimit` → `isLimit_of_principal_mul`
- `opow_principal_add_of_principal_add` → `principal_add_opow_of_principal_add`
- `mul_principal_add_is_principal_add` → `principal_add_mul_of_principal_add`
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Nov 1, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title refactor(SetTheory/Ordinal/Principal): fix bad names [Merged by Bors] - refactor(SetTheory/Ordinal/Principal): fix bad names Nov 1, 2024
@mathlib-bors mathlib-bors bot closed this Nov 1, 2024
@mathlib-bors mathlib-bors bot deleted the vi.princ_renames branch November 1, 2024 04:08
grunweg pushed a commit that referenced this pull request Nov 2, 2024
This PR performs the following renames:

- `principal_add_isLimit` → `isLimit_of_principal_add`
- `principal_mul_isLimit` → `isLimit_of_principal_mul`
- `opow_principal_add_of_principal_add` → `principal_add_opow_of_principal_add`
- `mul_principal_add_is_principal_add` → `principal_add_mul_of_principal_add`
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). t-set-theory Set theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants