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

Properly parse test cases with trailing whitespace #159

Merged
merged 1 commit into from Sep 1, 2015

Conversation

@nox
Copy link
Member

nox commented Aug 19, 2015

Review on Reviewable

@nox
Copy link
Member Author

nox commented Sep 1, 2015

This one will get bit-rotted by #158.

@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2015

The latest upstream changes (presumably #158) made this pull request unmergeable. Please resolve the merge conflicts.

@nox nox force-pushed the nox:whitespace-eof branch from 92a7287 to aace46f Sep 1, 2015
@nox
Copy link
Member Author

nox commented Sep 1, 2015

Fixed.

@SimonSapin
Copy link
Member

SimonSapin commented Sep 1, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2015

📌 Commit aace46f has been approved by SimonSapin

@bors-servo
Copy link
Contributor

bors-servo commented Sep 1, 2015

Testing commit aace46f with merge 2b9f1bc...

bors-servo pushed a commit that referenced this pull request Sep 1, 2015
Properly parse test cases with trailing whitespace



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

bors-servo commented Sep 1, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit aace46f into servo:master Sep 1, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@nox nox deleted the nox:whitespace-eof branch Sep 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.