Skip to content
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

Add the clippy/rust verison of lint introduced to the website #6492

Closed
xFrednet opened this issue Dec 21, 2020 · 1 comment · Fixed by #7813
Closed

Add the clippy/rust verison of lint introduced to the website #6492

xFrednet opened this issue Dec 21, 2020 · 1 comment · Fixed by #7813
Assignees
Labels
A-website Area: Improving the clippy website C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages

Comments

@xFrednet
Copy link
Member

xFrednet commented Dec 21, 2020

It might be useful to include the Rust and Clippy version that a lint was introduces on the lint list. This feature is useful for two reasons:

  1. It can happen that users pinned the compiler version for a project. (I believe that this is more common in other languages as Rust has a high compatibility grantee). It might than be interesting to know if a lint is already available for the compiler version they are using.
  2. The current README links to the lint list of master this already includes lints that are not included in the current stable rust version. Displaying the introduction version could reduce confusion

The versions should in my opinion be a part of the lint data (lints.json) for the website as loading the version dynamically would take up unnecessary performance.

@rustbot label +L-website, +C-needs-discussion

@xFrednet xFrednet changed the title Add the clippy/rust verison that a lint was introduced to the website Add the clippy/rust verison of lint introduced to the website Dec 21, 2020
@rustbot rustbot added S-needs-discussion Status: Needs further discussion before merging or work can be started A-website Area: Improving the clippy website labels Dec 21, 2020
@xFrednet
Copy link
Member Author

@rustbot claim

@xFrednet xFrednet added C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages and removed S-needs-discussion Status: Needs further discussion before merging or work can be started labels Oct 12, 2021
@bors bors closed this as completed in 3bfe98d Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-website Area: Improving the clippy website C-enhancement Category: Enhancement of lints, like adding more cases or adding help messages
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants