-
Notifications
You must be signed in to change notification settings - Fork 301
Add "Announcing Clippy feature freeze post #1613
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
base: master
Are you sure you want to change the base?
Changes from 2 commits
39fb959
4a0c77f
8cc70cf
20bdc4d
297b6fd
2c60a71
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 | ||||
---|---|---|---|---|---|---|
@@ -0,0 +1,62 @@ | ||||||
+++ | ||||||
path = "9999/12/31/the-incoming-summer-of-clippy" | ||||||
title = "The incoming Summer of Clippy" | ||||||
flip1995 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
authors = ["blyxyas"] | ||||||
|
||||||
[extra] | ||||||
team = "the Clippy team" | ||||||
team_url = "https://www.rust-lang.org/governance/teams/dev-tools#team-clippy" | ||||||
+++ | ||||||
|
||||||
We are currently working on a feature freeze for the Clippy project. This | ||||||
period would be 12 weeks long, starting on the introduction of 1.89 to beta | ||||||
on the 26 of June, and ending on the 18th of September. | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
This *summer of Clippy* comes from a lack of the necessary capacity needed | ||||||
to maintain all the current lints (about 750 of them 😱) and still add new ones. | ||||||
flip1995 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
We need to care for the Clippy project the same way that Clippy cares about our | ||||||
code, and note that every single one of them needs maintaining and accounting | ||||||
for current and future changes in the compiler as a whole. | ||||||
|
||||||
As a team we’ll focus on making contributing the best it can be, we’ll try to | ||||||
bisect all the issues possible and mark the issues that could be great as a | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
starting point to get familiar with the project. | ||||||
|
||||||
## What can I expect as a user? | ||||||
|
||||||
As a user you can expect to have more accurate lints, with less false negatives | ||||||
and more edge cases covered. A bump into Clippy’s toolchain would not involve | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
the same risk for false positives as it has been until now. We hope that by | ||||||
upping the standard for lints all around the project we can allow users to | ||||||
expect even more from the already great linter. | ||||||
|
||||||
If you have a lint which you want to be implemented, you can still open an | ||||||
issue for it, but no one will implement it (also, lint-adding pull requests | ||||||
opened after the starting date will be blocked for the duration of the period). | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
Note that pull requests opened before the starting date can still be reviewed | ||||||
and merged, but unless they are already in a very advanced state they will not | ||||||
be prioritized over bugfixes. | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
## Can I help? | ||||||
|
||||||
As always, any help on the project is very appreciated. Apart from opening bug | ||||||
reports, you can follow our [contributing guidelines] and check [our suggested | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
good first issues] for opening pull requests . Everything from fixing diagnostic | ||||||
flip1995 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
issues and improving documentation, to fixing application-wide crashes | ||||||
really helps Clippy get into the excellent state that our users deserve. | ||||||
|
||||||
If you need direct help when fixing a bug, don't hesitate to open a thread on | ||||||
Zulip, or to open the PR with the questions you have in order to start the | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
feedback stream. | ||||||
blyxyas marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
|
||||||
## Conclusion | ||||||
|
||||||
Thanks a lot for all these years of supporting Clippy, we hope that via this | ||||||
little feature hiatus we can achieve the excellence that the Rust user base | ||||||
deserves (if that's even possible). | ||||||
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.
Suggested change
Don't sell us short; Clippy is great! 🎉 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. It was more about how to Rust community is so good that they deserve something that nobody can deliver (but I suppose that by using Rust we are able to deliver on that promise?). 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. We can have nice things. |
||||||
|
||||||
[contributing guidelines]: https://github.com/rust-lang/rust-clippy/blob/master/CONTRIBUTING.md | ||||||
[our suggested good first issues]: <!-- ADD HERE OUR TRACKING ISSUE --> | ||||||
|
||||||
|
||||||
flip1995 marked this conversation as resolved.
Show resolved
Hide resolved
|
Uh oh!
There was an error while loading. Please reload this page.