Skip to content

Comments

Address some clippy lints.#295

Merged
bors-servo merged 1 commit intoservo:masterfrom
nical:clippy
Jun 8, 2018
Merged

Address some clippy lints.#295
bors-servo merged 1 commit intoservo:masterfrom
nical:clippy

Conversation

@nical
Copy link
Contributor

@nical nical commented Jun 7, 2018

I'd like to be able to run clippy every now and then but right now there is too much noise for me to notice anything. This removes most of the warnings down to a more manageable level.


This change is Reviewable

Copy link
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job, clippy!

@kvark
Copy link
Member

kvark commented Jun 8, 2018

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit b6c59fe has been approved by kvark

@bors-servo
Copy link
Contributor

⌛ Testing commit b6c59fe with merge 54a98c0...

bors-servo pushed a commit that referenced this pull request Jun 8, 2018
Address some clippy lints.

I'd like to be able to run clippy every now and then but right now there is too much noise for me to notice anything. This removes most of the warnings down to a more manageable level.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/euclid/295)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: kvark
Pushing 54a98c0 to master...

@bors-servo bors-servo merged commit b6c59fe into servo:master Jun 8, 2018
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