Skip to content

[Merged by Bors] - chore: remove unused @[simp] from Fin.sub_one_lt_iff#11967

Closed
kim-em wants to merge 1 commit intomasterfrom
sub_one_lt_iff
Closed

[Merged by Bors] - chore: remove unused @[simp] from Fin.sub_one_lt_iff#11967
kim-em wants to merge 1 commit intomasterfrom
sub_one_lt_iff

Conversation

@kim-em
Copy link
Copy Markdown
Contributor

@kim-em kim-em commented Apr 6, 2024

The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

@eric-wieser
Copy link
Copy Markdown
Member

Can you explain in the description why this lemma is bad?

@riccardobrasca
Copy link
Copy Markdown
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Apr 16, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 16, 2024
The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Apr 16, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: remove unused @[simp] from Fin.sub_one_lt_iff [Merged by Bors] - chore: remove unused @[simp] from Fin.sub_one_lt_iff Apr 16, 2024
@mathlib-bors mathlib-bors bot closed this Apr 16, 2024
@mathlib-bors mathlib-bors bot deleted the sub_one_lt_iff branch April 16, 2024 07:44
@kbuzzard
Copy link
Copy Markdown
Member

kbuzzard commented Apr 16, 2024

This lemma holds for Fin 0 as well, right? Oh! The statement doesn't typecheck if n=0 :-)

@riccardobrasca
Copy link
Copy Markdown
Member

I think so!

atarnoam pushed a commit that referenced this pull request Apr 16, 2024
The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
kim-em added a commit that referenced this pull request Apr 18, 2024
The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
callesonne pushed a commit that referenced this pull request Apr 22, 2024
The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Jun2M pushed a commit that referenced this pull request Apr 24, 2024
The @[simp] annotation is not used in mathlib, and triggered a linter failure that I don't care to investigate unnecessarily.

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

ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants