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

Tracking issue for RFC 2476, Clippy 1.0 #3343

Open
2 of 4 tasks
Manishearth opened this issue Oct 20, 2018 · 6 comments
Open
2 of 4 tasks

Tracking issue for RFC 2476, Clippy 1.0 #3343

Manishearth opened this issue Oct 20, 2018 · 6 comments
Labels
C-tracking-issue Category: Tracking Issue
Milestone

Comments

@Manishearth
Copy link
Member

Manishearth commented Oct 20, 2018

This is a tracking issue for the RFC "Clippy 1.0" (rust-lang/rfcs#2476).

Steps:

  • shadow_unrelated should be moved from restriction to pedantic
  • explicit_iter_loop and explicit_into_iter_loop be moved from style to pedantic
  • Add a version flag to Clippy that says 1.0
  • Update clippy_dummy published crate to 1.0

Unresolved questions:

None. The question of uplifts has been split out of this RFC into rust-lang/rust#53224

There isn't a "stabilization" step here per se, though we may consider moving the clippy-preview rustup component to clippy cc @nrc

@phansch phansch added this to the 1.0 milestone Oct 21, 2018
@Manishearth
Copy link
Member Author

Looks like the lint levels were changed already, i vaguely recall this happening during the rfc.

Anyone want to add the version flag?

@Centril
Copy link
Contributor

Centril commented Oct 23, 2018

Should the repo itself be moved to the rust-lang organization now to make it more official?

@Manishearth
Copy link
Member Author

Manishearth commented Oct 23, 2018 via email

@nrc
Copy link
Member

nrc commented Oct 24, 2018

I'd be happy for the repo to move to rust-lang, but I don't see a great rush. I plan to move rustfmt at some point after the 1.0 release, probably as soon as I have time to do it.

@oli-obk
Copy link
Contributor

oli-obk commented Oct 24, 2018

during such a move we could also drop the rust- prefix on the repo name

@phansch phansch added the C-tracking-issue Category: Tracking Issue label Dec 11, 2018
@retep998
Copy link
Member

retep998 commented Oct 7, 2020

The repo was moved to rust-lang but the rust- prefix was not dropped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-tracking-issue Category: Tracking Issue
Projects
None yet
Development

No branches or pull requests

6 participants