Skip to content

Conversation

flip1995
Copy link
Member

Fixes #3874

Checking for metadata wasn't enough anymore. --cap-lints allow only appears when compiling deps though.

changelog: none

@flip1995 flip1995 added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Nov 25, 2019
@bors
Copy link
Contributor

bors commented Nov 28, 2019

☔ The latest upstream changes (presumably #4848) made this pull request unmergeable. Please resolve the merge conflicts.

@phansch
Copy link
Contributor

phansch commented Nov 28, 2019

r=me with rebase

@flip1995
Copy link
Member Author

@bors r=phansch

@bors
Copy link
Contributor

bors commented Nov 28, 2019

📌 Commit a61fd43 has been approved by phansch

@bors
Copy link
Contributor

bors commented Nov 28, 2019

⌛ Testing commit a61fd43 with merge a4aa16d...

bors added a commit that referenced this pull request Nov 28, 2019
Don't error on clippy.toml of dependencies

Fixes #3874

Checking for `metadata` wasn't enough anymore. `--cap-lints allow` only appears when compiling deps though.

changelog: none
@flip1995 flip1995 added S-waiting-on-bors Status: The marked PR was approved and is only waiting bors and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties labels Nov 28, 2019
@bors
Copy link
Contributor

bors commented Nov 28, 2019

💔 Test failed - status-appveyor

@flip1995
Copy link
Member Author

@bors r=phansch

@bors
Copy link
Contributor

bors commented Nov 29, 2019

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: RIIR: Integration tests #4837

@bors
Copy link
Contributor

bors commented Nov 29, 2019

📌 Commit a61fd43 has been approved by phansch

bors added a commit that referenced this pull request Nov 29, 2019
Don't error on clippy.toml of dependencies

Fixes #3874

Checking for `metadata` wasn't enough anymore. `--cap-lints allow` only appears when compiling deps though.

changelog: none
@bors
Copy link
Contributor

bors commented Nov 29, 2019

⌛ Testing commit a61fd43 with merge 350f3d6...

@bors
Copy link
Contributor

bors commented Nov 29, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: phansch
Pushing 350f3d6 to master...

@bors bors merged commit a61fd43 into rust-lang:master Nov 29, 2019
@flip1995 flip1995 deleted the clippy_toml_deps branch May 4, 2020 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: The marked PR was approved and is only waiting bors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error reading Clippy's configuration file...
3 participants