Skip to content

[Merged by Bors] - feat(lint-style): rewrite the 'broad imports' linters in Lean#14059

Closed
grunweg wants to merge 4 commits intomasterfrom
MR-rewrite-broadImport
Closed

[Merged by Bors] - feat(lint-style): rewrite the 'broad imports' linters in Lean#14059
grunweg wants to merge 4 commits intomasterfrom
MR-rewrite-broadImport

Conversation

@grunweg
Copy link
Copy Markdown
Contributor

@grunweg grunweg commented Jun 23, 2024

This time, the error codes are intentionally changed, as the old ones feel actively misleading.


Open in Gitpod

grunweg added 3 commits June 23, 2024 20:45
Also remove the last ERR_OPT reference: that lint is not a syntax linter,
so this can safely be removed.
This time, the error codes are intentionally changed, as the old ones feel actively misleading.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 23, 2024

PR summary c0c8209c41

Import changes

No significant changes to the import graph


Declarations diff

+ BroadImports
+ broadImportsLinter

You can run this locally as follows
## summary with just the declaration names:
./scripts/no_lost_declarations.sh short <optional_commit>

## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit>

@ghost ghost added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jun 23, 2024
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jun 24, 2024
@ghost
Copy link
Copy Markdown

ghost commented Jun 24, 2024

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Jun 24, 2024

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Jun 24, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jun 24, 2024
This time, the error codes are intentionally changed, as the old ones feel actively misleading.



Co-authored-by: Jeremy Tan Jie Rui <reddeloostw@gmail.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 24, 2024

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Jun 24, 2024
This time, the error codes are intentionally changed, as the old ones feel actively misleading.



Co-authored-by: Jeremy Tan Jie Rui <reddeloostw@gmail.com>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jun 24, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(lint-style): rewrite the 'broad imports' linters in Lean [Merged by Bors] - feat(lint-style): rewrite the 'broad imports' linters in Lean Jun 24, 2024
@mathlib-bors mathlib-bors bot closed this Jun 24, 2024
@mathlib-bors mathlib-bors bot deleted the MR-rewrite-broadImport branch June 24, 2024 11:05
kbuzzard pushed a commit that referenced this pull request Jun 26, 2024
This time, the error codes are intentionally changed, as the old ones feel actively misleading.



Co-authored-by: Jeremy Tan Jie Rui <reddeloostw@gmail.com>
dagurtomas pushed a commit that referenced this pull request Jul 2, 2024
This time, the error codes are intentionally changed, as the old ones feel actively misleading.



Co-authored-by: Jeremy Tan Jie Rui <reddeloostw@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. t-linter Linter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants