Skip to content

/ core-graphics-rs

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 all the things #57

Merged
merged 6 commits into from Sep 1, 2016
Merged

Fix all the things #57

merged 6 commits into from Sep 1, 2016
+10 −11

Conversation

@nox
Copy link
Member

nox commented Sep 1, 2016 •
edited by larsbergstrom

This change is Reviewable
nox added 6 commits Sep 1, 2016
@nox
Silence all non_upper_case_globals warnings
39173f6
@nox
Remove some useless comment and commented-out code
17c0b1e
@nox
Remove some unused imports
e2e56e9
@nox
Silence an unused_variables warning
66df225
@nox
Don't reexport libc
e47505d
@nox
Fix all private_in_public errors
Loading status checks…
e805130
@Manishearth
Copy link
Member

Manishearth commented Sep 1, 2016

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

bors-servo commented Sep 1, 2016

📌 Commit e805130 has been approved by Manishearth
@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2016

⚡ Test exempted - status
@bors-servo bors-servo merged commit e805130 into servo:master Sep 1, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test exempted
Details
bors-servo added a commit that referenced this pull request Sep 1, 2016
@bors-servo
Auto merge of #57 - nox:fix-all-the-things, r=Manishearth
Loading status checks…
449990e 
Fix all the things

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-graphics-rs/57)
<!-- Reviewable:end -->
@bors-servo bors-servo mentioned this pull request Sep 1, 2016
Closed
@nox nox deleted the nox:fix-all-the-things branch Sep 1, 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
@nox @Manishearth @bors-servo
You can’t perform that action at this time.