Skip to content

Conversation

@krishna-veerareddy
Copy link
Contributor

changelog: none

@flip1995
Copy link
Member

Thanks!

This imports also appear in two tests. While LintPass is required in the first test, LintArray can be removed. From the second test, both imports can be removed.

tests/ui/lint_without_lint_pass.rs
9:use rustc_lint::{LintArray, LintPass};

tests/ui/default_lint.rs
9:use rustc_lint::{LintArray, LintPass};

@flip1995 flip1995 added the S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status) label Feb 11, 2020
@flip1995
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Feb 11, 2020

📌 Commit fcc3e72 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Feb 11, 2020

⌛ Testing commit fcc3e72 with merge e1c0a50...

bors added a commit that referenced this pull request Feb 11, 2020
…p1995

Remove unnecessary `Lint{Array|Pass}` imports

changelog: none
@bors
Copy link
Contributor

bors commented Feb 11, 2020

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing e1c0a50 to master...

@bors bors merged commit fcc3e72 into rust-lang:master Feb 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-author Status: This is awaiting some action from the author. (Use `@rustbot ready` to update this status)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants