-
Notifications
You must be signed in to change notification settings - Fork 1.8k
split clippy into lints, plugin and cargo-clippy #950
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,6 +9,7 @@ | |
|
||
# Generated by Cargo | ||
/target/ | ||
/clippy_lints/target/ | ||
|
||
# We don't pin yet | ||
Cargo.lock | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
[package] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Perhaps this should be an rlib? This would make compilation fast again. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure why that would speed it up. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oh, wait, no, the default cargo behavior would work here anyway. Nvm |
||
name = "clippy_lints" | ||
version = "0.0.69" | ||
authors = [ | ||
"Manish Goregaokar <[email protected]>", | ||
"Andre Bogus <[email protected]>", | ||
"Georg Brandl <[email protected]>", | ||
"Martin Carton <[email protected]>" | ||
] | ||
description = "A bunch of helpful lints to avoid common pitfalls in Rust" | ||
repository = "https://github.com/Manishearth/rust-clippy" | ||
readme = "README.md" | ||
license = "MPL-2.0" | ||
keywords = ["clippy", "lint", "plugin"] | ||
|
||
[dependencies] | ||
regex-syntax = "0.3.0" | ||
semver = "0.2.1" | ||
toml = "0.1" | ||
unicode-normalization = "0.1" | ||
quine-mc_cluskey = "0.2.2" | ||
|
||
[features] | ||
debugging = [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, it occurred to me that we might want to hardcode the version so as to avoid upgrade pains -- people shouldn't have to
cargo update -p clippy
and then still get errors because they didn't upgrade clippy_lints. Thoughts?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can add this to utils/update_lints.py so the version is automatically checked
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sweet. separate PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe someone who knows python should do that... or I can port it to Rust
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mcarton ?
Otherwise I'll fix it monday