Skip to content

[Merged by Bors] - feat: add uppercase lean 3 linter#1796

Closed
digama0 wants to merge 8 commits intomasterfrom
uppercase_lean3
Closed

[Merged by Bors] - feat: add uppercase lean 3 linter#1796
digama0 wants to merge 8 commits intomasterfrom
uppercase_lean3

Conversation

@digama0
Copy link
Copy Markdown
Member

@digama0 digama0 commented Jan 23, 2023

Implements a linter for lean 3 declarations containing capital letters (as suggested on Zulip).

@rwbarton
Copy link
Copy Markdown
Contributor

I'm going to add some logic to allow the most common Lean 3 name components with capital letters (Prop, Pi, and the intervals), since it is a bit much to add all the nolints otherwise.

@fpvandoorn
Copy link
Copy Markdown
Member

bors merge
bors d+ (if there is a conflict)

@bors
Copy link
Copy Markdown

bors bot commented Jan 25, 2023

✌️ digama0 can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Jan 25, 2023
bors bot pushed a commit that referenced this pull request Jan 25, 2023
* [ ] depends on: #1794

Implements a linter for lean 3 declarations containing capital letters (as [suggested on Zulip](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/non-aligned.20definitions/near/323102645)).

Co-authored-by: Mario Carneiro <di.gama@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Jan 25, 2023

Build failed (retrying...):

@fpvandoorn
Copy link
Copy Markdown
Member

as expected, this has merge conflicts

bors r-

@bors
Copy link
Copy Markdown

bors bot commented Jan 25, 2023

Canceled.

@rwbarton
Copy link
Copy Markdown
Contributor

bors merge

bors bot pushed a commit that referenced this pull request Jan 26, 2023
* [ ] depends on: #1794

Implements a linter for lean 3 declarations containing capital letters (as [suggested on Zulip](https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/non-aligned.20definitions/near/323102645)).

Co-authored-by: Mario Carneiro <di.gama@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Jan 26, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: add uppercase lean 3 linter [Merged by Bors] - feat: add uppercase lean 3 linter Jan 26, 2023
@bors bors bot closed this Jan 26, 2023
@bors bors bot deleted the uppercase_lean3 branch January 26, 2023 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants