Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(data/int|nat/basic): add add_monoid_hom.ext_nat/int#2957

Closed
urkud wants to merge 4 commits intomasterfrom
nat-ext
Closed

[Merged by Bors] - feat(data/int|nat/basic): add add_monoid_hom.ext_nat/int#2957
urkud wants to merge 4 commits intomasterfrom
nat-ext

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented Jun 5, 2020



/-- Two additive monoid homomorphisms `f`, `g` from `ℤ` to an additive monoid are equal
if `f 1 = g 1`. -/
@[ext] theorem ext_int [add_monoid A] {f g : ℤ →+ A} (h1 : f 1 = g 1) : f = g :=
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.

Should this be done for arbitrary cyclic groups first?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I would prefer to say something like "the subgroup closure {1} in int is top, so we're done" but data/int/basic is a dependency of submonoid.

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.

Ok, then we just prove it twice. It's not like it's that much work. I'll review this again. If I have no more comments, we can merge this.

Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Jun 5, 2020
bors bot pushed a commit that referenced this pull request Jun 5, 2020
@bors
Copy link
Copy Markdown

bors bot commented Jun 5, 2020

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(data/int|nat/basic): add add_monoid_hom.ext_nat/int [Merged by Bors] - feat(data/int|nat/basic): add add_monoid_hom.ext_nat/int Jun 5, 2020
@bors bors bot closed this Jun 5, 2020
@bors bors bot deleted the nat-ext branch June 5, 2020 10:00
cipher1024 pushed a commit to cipher1024/mathlib that referenced this pull request Mar 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants