Skip to content

[Merged by Bors] - chore: add induction_eliminator, cases_eliminator to WithTop etc#13264

Closed
urkud wants to merge 4 commits intomasterfrom
YK-induction-option
Closed

[Merged by Bors] - chore: add induction_eliminator, cases_eliminator to WithTop etc#13264
urkud wants to merge 4 commits intomasterfrom
YK-induction-option

Conversation

@urkud
Copy link
Copy Markdown
Member

@urkud urkud commented May 27, 2024

Add @[elab_as_elim, induction_eliminator, cases_eliminator] to

  • WithOne.recOneCoe (added cases_eliminator)
  • WithZero.recZeroCoe (added cases_eliminator)
  • ENNReal.recTopCoe
  • ENat.recTopCoe
  • EReal.rec

Mostly cherry-picked from #12605

Co-authored-by: @negiizhao


Open in Gitpod

Add `@[elab_as_elim, induction_eliminator, cases_eliminator]` to

- `WithBot.recBotCoe` (added `cases_eliminator`)
- `WithTop.recTopCoe` (added `cases_eliminator`)
- `ENNReal.recTopCoe`
- `ENat.recTopCoe`
- `EReal.rec`
@urkud urkud added awaiting-review awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. labels May 27, 2024
@urkud
Copy link
Copy Markdown
Member Author

urkud commented May 27, 2024

As a side effect, cases a on WithTop/WithBot uses top/bot instead of none. Reverted this change to keep this PR small.

@github-actions github-actions bot removed the awaiting-CI This PR does not pass CI yet. This label is automatically removed once it does. label May 27, 2024
@eric-wieser
Copy link
Copy Markdown
Member

As a side effect, cases a on WithTop/WithBot uses top/bot instead of none. Reverted this change to keep this PR small.

I think this is #13128?

@urkud
Copy link
Copy Markdown
Member Author

urkud commented May 27, 2024

As a side effect, cases a on WithTop/WithBot uses top/bot instead of none. Reverted this change to keep this PR small.

I think this is #13128?

Indeed, you're right! Thank you! There are too many open PRs at the moment, hard to keep track...

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

Looking again this doesn't overlap with #13128 after all.

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels May 27, 2024
mathlib-bors bot pushed a commit that referenced this pull request May 27, 2024
…13264)

Add `@[elab_as_elim, induction_eliminator, cases_eliminator]` to

- `WithOne.recOneCoe` (added `cases_eliminator`)
- `WithZero.recZeroCoe` (added `cases_eliminator`)
- `ENNReal.recTopCoe`
- `ENat.recTopCoe`
- `EReal.rec`

Mostly cherry-picked from #12605 

Co-authored-by: @negiizhao
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented May 27, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: add induction_eliminator, cases_eliminator to WithTop etc [Merged by Bors] - chore: add induction_eliminator, cases_eliminator to WithTop etc May 27, 2024
@mathlib-bors mathlib-bors bot closed this May 27, 2024
@mathlib-bors mathlib-bors bot deleted the YK-induction-option branch May 27, 2024 15:31
callesonne pushed a commit that referenced this pull request Jun 4, 2024
…13264)

Add `@[elab_as_elim, induction_eliminator, cases_eliminator]` to

- `WithOne.recOneCoe` (added `cases_eliminator`)
- `WithZero.recZeroCoe` (added `cases_eliminator`)
- `ENNReal.recTopCoe`
- `ENat.recTopCoe`
- `EReal.rec`

Mostly cherry-picked from #12605 

Co-authored-by: @negiizhao
js2357 pushed a commit that referenced this pull request Jun 18, 2024
…13264)

Add `@[elab_as_elim, induction_eliminator, cases_eliminator]` to

- `WithOne.recOneCoe` (added `cases_eliminator`)
- `WithZero.recZeroCoe` (added `cases_eliminator`)
- `ENNReal.recTopCoe`
- `ENat.recTopCoe`
- `EReal.rec`

Mostly cherry-picked from #12605 

Co-authored-by: @negiizhao
mathlib-bors bot pushed a commit that referenced this pull request Jun 30, 2024
These are no longer needed after #13264
dagurtomas pushed a commit that referenced this pull request Jul 2, 2024
These are no longer needed after #13264
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.

2 participants