Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: remove triv tactic #712

Merged
merged 2 commits into from Apr 1, 2024
Merged

chore: remove triv tactic #712

merged 2 commits into from Apr 1, 2024

Conversation

semorrison
Copy link
Collaborator

This is not used in Mathlib; all uses can just become trivial. Let's remove the unnecessary duplication.

leanprover-community/mathlib4#11679

@semorrison semorrison added the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Mar 26, 2024
@digama0 digama0 added will-merge-soon PR will be merged soon. Any concerns should be raised quickly. and removed awaiting-review This PR is ready for review; the author thinks it is ready to be merged. labels Mar 26, 2024
@digama0
Copy link
Member

digama0 commented Mar 26, 2024

approved pending merge of the mathlib4 PR.

mathlib-bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 26, 2024
Std defines `triv`, a slight variation on `trivial`. It appears that Mathlib doesn't care about the distinction (any more?) and so we can consolidate on a single tactic.

leanprover-community/batteries#712 separately replaces `triv` in Std with an error explaining to use `trivial`.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@joehendrix joehendrix merged commit 67c40f8 into main Apr 1, 2024
2 checks passed
@joehendrix joehendrix deleted the rm_triv branch April 1, 2024 20:45
Louddy pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 15, 2024
Std defines `triv`, a slight variation on `trivial`. It appears that Mathlib doesn't care about the distinction (any more?) and so we can consolidate on a single tactic.

leanprover-community/batteries#712 separately replaces `triv` in Std with an error explaining to use `trivial`.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
atarnoam pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 16, 2024
Std defines `triv`, a slight variation on `trivial`. It appears that Mathlib doesn't care about the distinction (any more?) and so we can consolidate on a single tactic.

leanprover-community/batteries#712 separately replaces `triv` in Std with an error explaining to use `trivial`.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
uniwuni pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 19, 2024
Std defines `triv`, a slight variation on `trivial`. It appears that Mathlib doesn't care about the distinction (any more?) and so we can consolidate on a single tactic.

leanprover-community/batteries#712 separately replaces `triv` in Std with an error explaining to use `trivial`.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
callesonne pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 22, 2024
Std defines `triv`, a slight variation on `trivial`. It appears that Mathlib doesn't care about the distinction (any more?) and so we can consolidate on a single tactic.

leanprover-community/batteries#712 separately replaces `triv` in Std with an error explaining to use `trivial`.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
will-merge-soon PR will be merged soon. Any concerns should be raised quickly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants