Skip to content

[Merged by Bors] - chore(GroupTheory/Coset): deprecate some lemma#15482

Closed
astrainfinita wants to merge 3 commits intomasterfrom
FR_quotientgroup_defeqabuse
Closed

[Merged by Bors] - chore(GroupTheory/Coset): deprecate some lemma#15482
astrainfinita wants to merge 3 commits intomasterfrom
FR_quotientgroup_defeqabuse

Conversation

@astrainfinita
Copy link
Copy Markdown
Collaborator

@astrainfinita astrainfinita commented Aug 4, 2024

They were different in mathlib3 because of coercions.


Open in Gitpod

@astrainfinita astrainfinita added the t-algebra Algebra (groups, rings, fields, etc) label Aug 4, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 4, 2024

PR summary 138c3ae4dd

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.

@astrainfinita astrainfinita changed the title chore(GroupTheory/QuotientGroup): reduce defeq abuse chore(GroupTheory/Coset): deprecate some lemma Aug 4, 2024
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 merge

Thanks!

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Aug 5, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 5, 2024
They were different in mathlib3 because of coercions.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 5, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(GroupTheory/Coset): deprecate some lemma [Merged by Bors] - chore(GroupTheory/Coset): deprecate some lemma Aug 5, 2024
@mathlib-bors mathlib-bors bot closed this Aug 5, 2024
@mathlib-bors mathlib-bors bot deleted the FR_quotientgroup_defeqabuse branch August 5, 2024 13:16
@[to_additive (attr := elab_as_elim)]
theorem induction_on' {C : α ⧸ s → Prop} (x : α ⧸ s) (H : ∀ z : α, C z) : C x :=
Quotient.inductionOn' x H
@[to_additive (attr := deprecated (since := "2024-08-04"))] alias induction_on' := induction_on
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I know this has been merged now but did we just lose an elab_as_elim attribute on the primed declaration?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch, though it probably doesn't matter much since this is now deprecated.

bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
They were different in mathlib3 because of coercions.
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
They were different in mathlib3 because of coercions.
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
They were different in mathlib3 because of coercions.
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.

3 participants