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

Stabilize workspace wrapper. #6687

Merged
merged 1 commit into from
Feb 10, 2021
Merged

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Feb 6, 2021

This fixes it so that cargo clippy doesn't share the same cache artifacts as cargo check. The Cargo side was stabilized a while ago (rust-lang/cargo#8976), so this should be ready to go. I'm not aware of any blockers or concerns.

Closes #4612


changelog: cargo clippy no longer shares the same build cache as cargo check.

@rust-highfive
Copy link

r? @flip1995

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 6, 2021
@flip1995
Copy link
Member

@bors r+

Thanks! This closes our first project on our 2021 roadmap!

@bors
Copy link
Collaborator

bors commented Feb 10, 2021

📌 Commit 0585c34 has been approved by flip1995

@bors
Copy link
Collaborator

bors commented Feb 10, 2021

⌛ Testing commit 0585c34 with merge 9c0ae2a...

@bors
Copy link
Collaborator

bors commented Feb 10, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995
Pushing 9c0ae2a to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clippy doesn't report error/warnings after cargo check
4 participants