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

Update BARE_TRAIT_OBJECTS lint to deny in 2021 edition #81244

Closed
wants to merge 1 commit into from

Conversation

rylev
Copy link
Member

@rylev rylev commented Jan 21, 2021

As part of #80165 this upgrades the BARE_TRAIT_OBJECTS lint from Warn to Deny in the 2021 edition. In practice this doesn't change very much as the compiler is still compiled in 2018 edition.

There is active discussion on whether a deny by default lint is the right path or whether this should be handled in another part of the compiler as a hard error. This PR can be used to further that discussion.

r? @nikomatsakis

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 21, 2021
@jyn514 jyn514 added D-edition Diagnostics: An error or lint that should account for edition differences. T-lang Relevant to the language team, which will review and decide on the PR/issue. labels Jan 21, 2021
@crlf0710 crlf0710 added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 5, 2021
@nikomatsakis
Copy link
Contributor

@rylev -- do you plan to open a consolidated pr? should we close this one for now?

@rylev
Copy link
Member Author

rylev commented Mar 9, 2021

Yep, let's close this for now.

@rylev rylev closed this Mar 9, 2021
@rylev rylev deleted the upgrade-bare-trait-to-error branch March 9, 2021 18:28
bors added a commit to rust-lang-ci/rust that referenced this pull request May 4, 2021
…atsakis

Update BARE_TRAIT_OBJECT and ELLIPSIS_INCLUSIVE_RANGE_PATTERNS to errors in Rust 2021

This addresses rust-lang#81244 by updating two lints to errors in the Rust 2021 edition.

r? `@estebank`
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request May 6, 2021
Update BARE_TRAIT_OBJECT and ELLIPSIS_INCLUSIVE_RANGE_PATTERNS to errors in Rust 2021

This addresses rust-lang/rust#81244 by updating two lints to errors in the Rust 2021 edition.

r? `@estebank`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-edition Diagnostics: An error or lint that should account for edition differences. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-lang Relevant to the language team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants