Skip to content

Comments

[Merged by Bors] - chore: classify avoid simpNF linter error porting notes#10760

Closed
pitmonticone wants to merge 4 commits intomasterfrom
pitmonticone/avoid-linter-error
Closed

[Merged by Bors] - chore: classify avoid simpNF linter error porting notes#10760
pitmonticone wants to merge 4 commits intomasterfrom
pitmonticone/avoid-linter-error

Conversation

@pitmonticone
Copy link
Member

Classifies by adding issue number (#10759) to porting notes claiming:

removed @[simp] to avoid a simpNF linter error.

@pitmonticone pitmonticone added awaiting-review easy < 20s of review time. See the lifecycle page for guidelines. porting-notes Mathlib3 to Mathlib4 porting notes. labels Feb 20, 2024
Copy link
Contributor

@Vierkantor Vierkantor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these notes should be classified the same as #10619.

@Vierkantor Vierkantor added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Feb 20, 2024
Copy link
Member Author

@pitmonticone pitmonticone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Transferred into #10619.

Copy link
Contributor

@Vierkantor Vierkantor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since that PR got approved, let's get this merged too!

bors r+

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Feb 20, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 20, 2024
Classifies by adding issue number (#10759) to porting notes claiming: 

> removed `@[simp]` to avoid a `simpNF` linter error.
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Feb 20, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: classify avoid simpNF linter error porting notes [Merged by Bors] - chore: classify avoid simpNF linter error porting notes Feb 20, 2024
@mathlib-bors mathlib-bors bot closed this Feb 20, 2024
@mathlib-bors mathlib-bors bot deleted the pitmonticone/avoid-linter-error branch February 20, 2024 23:51
thorimur pushed a commit that referenced this pull request Feb 24, 2024
Classifies by adding issue number (#10759) to porting notes claiming: 

> removed `@[simp]` to avoid a `simpNF` linter error.
thorimur pushed a commit that referenced this pull request Feb 26, 2024
Classifies by adding issue number (#10759) to porting notes claiming: 

> removed `@[simp]` to avoid a `simpNF` linter error.
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
Classifies by adding issue number (#10759) to porting notes claiming: 

> removed `@[simp]` to avoid a `simpNF` linter error.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
Classifies by adding issue number (#10759) to porting notes claiming: 

> removed `@[simp]` to avoid a `simpNF` linter error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

easy < 20s of review time. See the lifecycle page for guidelines. porting-notes Mathlib3 to Mathlib4 porting notes. 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