Skip to content

/ euclid

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

Various cleanup. #118

Merged
merged 4 commits into from Jan 13, 2016
Merged

Various cleanup. #118

merged 4 commits into from Jan 13, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Jan 13, 2016

Review on Reviewable
Ms2ger added 4 commits Jan 13, 2016
@Ms2ger
Remove unused imports.
4d2c7ab
@Ms2ger
Use Zero and One from num, not std::num.
db68b61
@Ms2ger
Limit the dependency on rand to tests.
0889021
@Ms2ger
Fix a build warning in the tests.
Loading status checks…
fdb6c5a
@frewsxcv
Copy link
Member

frewsxcv commented Jan 13, 2016

@bors-servo r+
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

📌 Commit fdb6c5a has been approved by frewsxcv
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

⌛ Testing commit fdb6c5a with merge 32d20fa...
bors-servo added a commit that referenced this pull request Jan 13, 2016
@bors-servo
Auto merge of #118 - servo:various, r=frewsxcv
Loading status checks…
32d20fa 
Various cleanup.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/euclid/118)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

☀️ Test successful - travis
@bors-servo bors-servo merged commit fdb6c5a into master Jan 13, 2016
3 checks passed
3 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@Ms2ger Ms2ger deleted the various branch Jan 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@Ms2ger @frewsxcv @bors-servo
You can’t perform that action at this time.