Skip to content

[Merged by Bors] - feat: nontriviality tactic#600

Closed
digama0 wants to merge 6 commits intomasterfrom
nontriviality
Closed

[Merged by Bors] - feat: nontriviality tactic#600
digama0 wants to merge 6 commits intomasterfrom
nontriviality

Conversation

@digama0
Copy link
Copy Markdown
Member

@digama0 digama0 commented Nov 14, 2022

Supercedes #548.

@kim-em kim-em added the blocked-by-core-PR This PR depends on a PR to Core/Std label Nov 16, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 16, 2022

CI failures are caused by the issue reported and fixed at leanprover/lean4#1829. We'll need to bump, but bumping to 2022-11-16 is blocked by leanprover/lean4#1842.

@Ruben-VandeVelde
Copy link
Copy Markdown
Contributor

Merging master fixes the build; just some missing documentation now

-- Mathlib.Tactic.Nontriviality.Core
#check Mathlib.Tactic.Nontriviality.Parser.Attr.nontriviality /- definition missing documentation string -/
#check Mathlib.Tactic.Nontriviality.nontrivialityByAssumption /- definition missing documentation string -/
#check Mathlib.Tactic.Nontriviality.nontrivialityByElim /- definition missing documentation string -/
#check Mathlib.Tactic.Nontriviality.elabNontriviality /- definition missing documentation string -/

@Ruben-VandeVelde Ruben-VandeVelde added awaiting-author A reviewer has asked the author a question or requested changes. and removed blocked-by-core-PR This PR depends on a PR to Core/Std labels Nov 18, 2022
@Ruben-VandeVelde Ruben-VandeVelde added awaiting-review and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Nov 19, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 20, 2022

bors merge

@bors
Copy link
Copy Markdown

bors bot commented Nov 20, 2022

Merge conflict.

@digama0
Copy link
Copy Markdown
Member Author

digama0 commented Nov 21, 2022

bors merge

@digama0
Copy link
Copy Markdown
Member Author

digama0 commented Nov 21, 2022

bors ping

@bors
Copy link
Copy Markdown

bors bot commented Nov 21, 2022

pong

@digama0
Copy link
Copy Markdown
Member Author

digama0 commented Nov 21, 2022

bors r+

bors bot pushed a commit that referenced this pull request Nov 21, 2022
Supercedes #548.

Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Nov 21, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: nontriviality tactic [Merged by Bors] - feat: nontriviality tactic Nov 21, 2022
@bors bors bot closed this Nov 21, 2022
@bors bors bot deleted the nontriviality branch November 21, 2022 03:57
* (not implemented yet) `solve_by_elim with attr₁ ... attrᵣ` also applies all lemmas tagged with
the specified attributes.
* `solve_by_elim [h₁, h₂, ..., hᵣ, attr₁, ... attrᵣ]` also applies the named lemmas, as well as
all lemmas tagged with the specified attributes.
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.

@digama0 has support been added to solve_by_elim so that it now with with attributes like this? If not, this comment should be marked "not implemented yet".

bors bot pushed a commit that referenced this pull request Nov 23, 2022
mathlib hash: 7cca171008afb30576d2d4c51173700a780c23d0

- [x] depends on: #600 

Co-authored-by: Mario Carneiro <di.gama@gmail.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Yakov Pechersky <ffxen158@gmail.com>
Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants