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

Fix unused_must_use inside Box #806

Merged
merged 1 commit into from
Jun 30, 2019
Merged

Fix unused_must_use inside Box #806

merged 1 commit into from
Jun 30, 2019

Conversation

varkor
Copy link
Member

@varkor varkor commented Jun 30, 2019

After rust-lang/rust#62228, this will be linted against (and causes the test to fail). (This blocks rust-lang/rust#62228.)

After rust-lang/rust#62228, this will be linted against (and causes the test to fail).
@RalfJung
Copy link
Member

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Jun 30, 2019

📌 Commit 5d4aae8 has been approved by RalfJung

bors added a commit that referenced this pull request Jun 30, 2019
Fix `unused_must_use` inside `Box`

After rust-lang/rust#62228, this will be linted against (and causes the test to fail). (This blocks rust-lang/rust#62228.)
@bors
Copy link
Contributor

bors commented Jun 30, 2019

⌛ Testing commit 5d4aae8 with merge 72b2e10...

@bors
Copy link
Contributor

bors commented Jun 30, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 72b2e10 to master...

@bors bors merged commit 5d4aae8 into rust-lang:master Jun 30, 2019
@varkor varkor deleted the patch-1 branch June 30, 2019 16:44
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.

3 participants