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

[Merged by Bors] - chore(algebra/*): add back nat_algebra_subsingleton and add_comm_monoid.nat_semimodule.subsingleton#7263

Closed
eric-wieser wants to merge 3 commits intomasterfrom
eric-wieser/nat-subsingleton
Closed

[Merged by Bors] - chore(algebra/*): add back nat_algebra_subsingleton and add_comm_monoid.nat_semimodule.subsingleton#7263
eric-wieser wants to merge 3 commits intomasterfrom
eric-wieser/nat-subsingleton

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser commented Apr 19, 2021

As suggested in #7084 (comment).

Even if we now have a design solution that makes this unnecessary, it still feels like a result worth stating.


Open in Gitpod

@eric-wieser eric-wieser requested a review from sgouezel April 19, 2021 09:55
@sgouezel
Copy link
Copy Markdown
Collaborator

I have already added them back in #7255 as was requested.

@eric-wieser
Copy link
Copy Markdown
Member Author

You haven't added back the subsingleton (algebra nat R) one in that PR. I'd assumed that PR was limited to int refactors, so wasn't expecting it to finish up some nat cleanup too.

@eric-wieser
Copy link
Copy Markdown
Member Author

I've updated to use the comments and lemma names from that PR, but was able to produce a shorter proof

@sgouezel
Copy link
Copy Markdown
Collaborator

Nice, thanks!
bors r+

@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 Apr 19, 2021
@bors
Copy link
Copy Markdown

bors bot commented Apr 19, 2021

Merge conflict.

@github-actions github-actions bot added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Apr 19, 2021
@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Apr 19, 2021
@eric-wieser
Copy link
Copy Markdown
Member Author

bors r+

bors bot pushed a commit that referenced this pull request Apr 20, 2021
…id.nat_semimodule.subsingleton (#7263)

As suggested in #7084 (comment).

Even if we now have a design solution that makes this unnecessary, it still feels like a result worth stating.
@bors
Copy link
Copy Markdown

bors bot commented Apr 20, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(algebra/*): add back nat_algebra_subsingleton and add_comm_monoid.nat_semimodule.subsingleton [Merged by Bors] - chore(algebra/*): add back nat_algebra_subsingleton and add_comm_monoid.nat_semimodule.subsingleton Apr 20, 2021
@bors bors bot closed this Apr 20, 2021
@bors bors bot deleted the eric-wieser/nat-subsingleton branch April 20, 2021 03:28
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