Skip to content

/ servo

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

Use wptrunner for some old-style reftests. #8793

Merged
merged 3 commits into from Dec 3, 2015
Merged

Use wptrunner for some old-style reftests. #8793

merged 3 commits into from Dec 3, 2015

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Dec 3, 2015

Review on Reviewable
Ms2ger added 3 commits Dec 2, 2015
@Ms2ger
Move text_overflow_a.html to wpt reftests.
1ff3424
@Ms2ger
Move text_decoration_propagation_a.html to wpt reftests.
d8fe60c
@Ms2ger
Move vertical-lr-blocks.html to wpt reftests.
Loading status checks…
81a5948
@frewsxcv frewsxcv self-assigned this Dec 3, 2015
@frewsxcv
Copy link
Member

frewsxcv commented Dec 3, 2015

@bors-servo r+

Reviewed 16 of 16 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io
@bors-servo
Copy link
Contributor

bors-servo commented Dec 3, 2015

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

bors-servo commented Dec 3, 2015

⌛ Testing commit 81a5948 with merge 93666ef...
bors-servo added a commit that referenced this pull request Dec 3, 2015
@bors-servo
Auto merge of #8793 - Ms2ger:test-ref, r=frewsxcv
93666ef 
Use wptrunner for some old-style reftests.

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

bors-servo commented Dec 3, 2015

💔 Test failed - linux-rel
@Ms2ger
Copy link
Contributor Author

Ms2ger commented Dec 3, 2015

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Dec 3, 2015

⌛ Testing commit 81a5948 with merge db0a0ac...
bors-servo added a commit that referenced this pull request Dec 3, 2015
@bors-servo
Auto merge of #8793 - Ms2ger:test-ref, r=frewsxcv
Loading status checks…
db0a0ac 
Use wptrunner for some old-style reftests.

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

bors-servo commented Dec 3, 2015

☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt
@bors-servo bors-servo merged commit 81a5948 into servo:master Dec 3, 2015
3 checks passed
3 checks passed
code-review/reviewable Review complete: all files reviewed, all discussions resolved
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Ms2ger Ms2ger deleted the Ms2ger:test-ref branch Dec 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@frewsxcv frewsxcv

Labels
S-awaiting-merge
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

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