Skip to content

Conversation

@samueltardieu
Copy link
Member

By default, lintcheck will use the clippy.toml file found at the toplevel of the repository (CARGO_MANIFEST_DIR). This file is meant for configuration of Clippy applied to Clippy sources.

This creates a new lintcheck/ci-config/clippy.toml file which is used by the CI when running lintcheck. By default this uses the default Clippy configuration.

changelog: none

@rustbot
Copy link
Collaborator

rustbot commented Feb 16, 2025

r? @Jarcho

rustbot has assigned @Jarcho.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 16, 2025
@samueltardieu
Copy link
Member Author

r? @flip1995
since it's related to the CI

@rustbot rustbot assigned flip1995 and unassigned Jarcho Feb 17, 2025
@flip1995
Copy link
Member

I wonder if we should enable avoid-breaking-exported-api = false, in that config file, so that we get even broader lint coverage. WDYT?

@samueltardieu
Copy link
Member Author

I wonder if we should enable avoid-breaking-exported-api = false, in that config file, so that we get even broader lint coverage. WDYT?

Let's try a run with it!

@samueltardieu
Copy link
Member Author

That didn't change anything for the default lintcheck run, we can let it in.

By default, lintcheck will use the `clippy.toml` file found at the
toplevel of the repository (`CARGO_MANIFEST_DIR`). This file is meant
for configuration of Clippy applied to Clippy sources.

This creates a new `lintcheck/ci-config/clippy.toml` file which is used
by the CI when running lintcheck. By default this uses the default
Clippy configuration.
@samueltardieu
Copy link
Member Author

Rebased

Copy link
Member

@flip1995 flip1995 left a comment

Choose a reason for hiding this comment

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

Completely forgot about this PR. Let's get this merged 👍

@flip1995 flip1995 added this pull request to the merge queue Mar 22, 2025
Merged via the queue into rust-lang:master with commit d5a6688 Mar 22, 2025
11 checks passed
@samueltardieu samueltardieu deleted the push-pxvyoozrxozs branch March 22, 2025 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants