Skip to content

[Merged by Bors] - fix: to_additive translates pow to nsmul#1502

Closed
fpvandoorn wants to merge 16 commits intomasterfrom
nsmul
Closed

[Merged by Bors] - fix: to_additive translates pow to nsmul#1502
fpvandoorn wants to merge 16 commits intomasterfrom
nsmul

Conversation

@fpvandoorn
Copy link
Copy Markdown
Member

@fpvandoorn fpvandoorn commented Jan 12, 2023

  • I tried translating it to smul in [Merged by Bors] - feat: improve toAdditive.guessName #715, but that was a bad decision
  • It is possible that some lemmas that want to be called smul now use nsmul. This doesn't raise an error unless they are aligned or explicitly used elsewhere.
  • Rename some lemmas from smul to nsmul.
  • Zulip

Open in Gitpod

@fpvandoorn fpvandoorn added the awaiting-author A reviewer has asked the author a question or requested changes. label Jan 12, 2023
@fpvandoorn fpvandoorn added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Jan 12, 2023
@fpvandoorn fpvandoorn added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Jan 12, 2023
@kim-em kim-em added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 12, 2023
@fpvandoorn fpvandoorn added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. and removed awaiting-author A reviewer has asked the author a question or requested changes. merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) labels Jan 23, 2023
@fpvandoorn fpvandoorn removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label Jan 24, 2023
@rwbarton
Copy link
Copy Markdown
Contributor

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Jan 24, 2023
bors bot pushed a commit that referenced this pull request Jan 24, 2023
* I tried translating it to `smul` in #715, but that was a bad decision
* It is possible that some lemmas that want to be called `smul` now use `nsmul`. This doesn't raise an error unless they are aligned or explicitly used elsewhere.
* Rename some lemmas from `smul` to `nsmul`.
* [Zulip](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/to_additive.20translating.20pow.20to.20nsmul)



Co-authored-by: Reid Barton <rwbarton@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Jan 24, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title fix: to_additive translates pow to nsmul [Merged by Bors] - fix: to_additive translates pow to nsmul Jan 24, 2023
@bors bors bot closed this Jan 24, 2023
@bors bors bot deleted the nsmul branch January 24, 2023 16:59
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants