Skip to content

Add a test for #2526#3811

Merged
bors merged 1 commit into
masterfrom
test-for-2526
Feb 24, 2019
Merged

Add a test for #2526#3811
bors merged 1 commit into
masterfrom
test-for-2526

Conversation

@mcarton

@mcarton mcarton commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.

@flip1995

Copy link
Copy Markdown
Member

Thanks!

@bors r+

@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

📌 Commit a05f305 has been approved by flip1995

@mcarton

mcarton commented Feb 24, 2019

Copy link
Copy Markdown
Contributor Author

I'm an idiot, I was fiddling with the attribute, to check for other crate type and forgot to restore it before committing 😅

@flip1995

flip1995 commented Feb 24, 2019

Copy link
Copy Markdown
Member

Oh I also completely missed that! Thanks for the fast fix. Now this should be good to go.

@bors r+

@bors

This comment has been minimized.

@bors

This comment has been minimized.

@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

📌 Commit 3a9426e has been approved by flip1995

@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit 3a9426e with merge f7fa2a8...

bors added a commit that referenced this pull request Feb 24, 2019
Add a test for #2526

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.
@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

💔 Test failed - checks-travis

@mcarton

mcarton commented Feb 24, 2019

Copy link
Copy Markdown
Contributor Author

@bors r+

@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

📌 Commit 8bcd546 has been approved by mcarton

@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

⌛ Testing commit 8bcd546 with merge e99611b...

bors added a commit that referenced this pull request Feb 24, 2019
Add a test for #2526

Closes #2526, which seems to have been fixed at some point, but I couldn't find a test for it.
@mcarton

mcarton commented Feb 24, 2019

Copy link
Copy Markdown
Contributor Author

The test had an extra blank line…

@bors

bors commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

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

@bors bors merged commit 8bcd546 into master Feb 24, 2019
@mcarton mcarton deleted the test-for-2526 branch February 24, 2019 18:55
@oli-obk

oli-obk commented Feb 24, 2019

Copy link
Copy Markdown
Contributor

Yea, you can r+ your own PRs. The convention is to r=actual_reviewer instead of r+

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.

Clippy does not run on proc macro crates

4 participants