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

Enable (and fix) extra lint groups required for in-tree build #12826

Merged
merged 4 commits into from
Jul 20, 2022

Conversation

fasterthanlime
Copy link
Contributor

This enables 3 lint groups that are required to build rust-analyzer as an "in-tree" (git subtree) tool in rust-lang/rust, and fixes all relevant diagnostics.

This change is tracked in:

Maintainer impact: more warnings, should be easy enough to fix them (it's mostly looking out for "rust-2015-isms", the lint group is poorly named). If you forget some, they'll show up during a ra=>rust sync.

@fasterthanlime fasterthanlime changed the title Enable extra warnings required by rust-lang/rust Enable (and fix) extra lint groups required for in-tree build Jul 20, 2022
@lnicola
Copy link
Member

lnicola commented Jul 20, 2022

Seems reasonable, but it will probably make every open PR unmergeable (at least we don't have a lot of them right now).

@lnicola
Copy link
Member

lnicola commented Jul 20, 2022

r? @Veykril

@fasterthanlime
Copy link
Contributor Author

Seems reasonable, but it will probably make every open PR unmergeable (at least we don't have a lot of them right now).

Yeah :( That's the only "touches 260+ files" PR tracked by #12818 - the rest is nice small incremental changes, so I wanted to get it out of the way early.

@Veykril
Copy link
Member

Veykril commented Jul 20, 2022

Seems reasonable, but it will probably make every open PR unmergeable (at least we don't have a lot of them right now).

Good thing I tried my best to get most merged over the past week

@Veykril
Copy link
Member

Veykril commented Jul 20, 2022

Can't believe rust_2018_idiom is still not warned by default tbh, also thanks a lot for putting so much effort into this. Really excited for these changes!
@bors r+

@bors
Copy link
Contributor

bors commented Jul 20, 2022

📌 Commit 7e285e1 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Jul 20, 2022

⌛ Testing commit 7e285e1 with merge 28bab68...

@lnicola
Copy link
Member

lnicola commented Jul 20, 2022

Good thing I tried my best to get most merged over the past week

Exactly :-).

Can't believe rust_2018_idiom is still not warned by default tbh

I think it should have been made into a warning in 2021.

@bors
Copy link
Contributor

bors commented Jul 20, 2022

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 28bab68 to master...

@bors bors merged commit 28bab68 into rust-lang:master Jul 20, 2022
@Veykril
Copy link
Member

Veykril commented Jul 20, 2022

I don't think this added any new conflicts? 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants