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

Remove the legacy reftest framework. #9293

Merged
merged 3 commits into from Jan 14, 2016
Merged

Remove the legacy reftest framework. #9293

merged 3 commits into from Jan 14, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Jan 13, 2016

Review on Reviewable
@Ms2ger Ms2ger force-pushed the Ms2ger:reftests branch from 3fa6306 to 03b9c13 Jan 13, 2016
@SimonSapin
Copy link
Member

SimonSapin commented Jan 13, 2016

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

bors-servo commented Jan 13, 2016

📌 Commit 03b9c13 has been approved by SimonSapin
@jdm jdm mentioned this pull request Jan 13, 2016
Closed
@frewsxcv
Copy link
Member

frewsxcv commented Jan 13, 2016

./python/tidy.py:64: F821 undefined name 'reftest_filetype'
@jdm
Copy link
Member

jdm commented Jan 13, 2016

@bors-servo: r-
@jdm jdm added S-fails-tidy and removed S-awaiting-merge labels Jan 13, 2016
@Ms2ger Ms2ger force-pushed the Ms2ger:reftests branch from 03b9c13 to 78e7196 Jan 13, 2016
@Ms2ger
Copy link
Contributor Author

Ms2ger commented Jan 13, 2016

@bors-servo r=SimonSapin
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

📌 Commit 78e7196 has been approved by SimonSapin
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2016

⌛ Testing commit 78e7196 with merge 7c8bb9f...
bors-servo added a commit that referenced this pull request Jan 14, 2016
@bors-servo
Auto merge of #9293 - Ms2ger:reftests, r=SimonSapin
7c8bb9f 
Remove the legacy reftest framework.

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

bors-servo commented Jan 14, 2016

💔 Test failed - linux-rel
@KiChjang
Copy link
Member

KiChjang commented Jan 14, 2016 

Ran 3837 tests finished in 264.0 seconds.
  • 3836 ran as expected. 701 tests skipped.
  • 1 tests timed out unexpectedly

Tests with unexpected results:
  ▶ TIMEOUT [expected PASS] /_mozilla/css/acid2-wrapper.html
@emilio emilio removed the S-fails-tidy label Jan 14, 2016
Ms2ger added 3 commits Jan 12, 2016
@Ms2ger
Move acid2.html to wpt reftests.
f7db33e
@Ms2ger
Remove legacy reftest lints.
671cfd9
@Ms2ger
Remove the legacy reftest framework.
Loading status checks…
d8511c8
@Ms2ger Ms2ger force-pushed the Ms2ger:reftests branch from 78e7196 to d8511c8 Jan 14, 2016
@Ms2ger
Copy link
Contributor Author

Ms2ger commented Jan 14, 2016

@bors-servo r=SimonSapin
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2016

📌 Commit d8511c8 has been approved by SimonSapin
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2016

⌛ Testing commit d8511c8 with merge d28fb42...
bors-servo added a commit that referenced this pull request Jan 14, 2016
@bors-servo
Auto merge of #9293 - Ms2ger:reftests, r=SimonSapin
Loading status checks…
d28fb42 
Remove the legacy reftest framework.

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

bors-servo commented Jan 14, 2016

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

Ms2ger commented Jan 14, 2016

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

bors-servo commented Jan 14, 2016

⚡ Previous build results for android, gonk, linux-dev, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt are reusable. Rebuilding only linux-rel...
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2016

☀️ Test successful - android, gonk, linux-dev, linux-rel, mac-dev-ref-unit, mac-rel-css, mac-rel-wpt
@bors-servo bors-servo merged commit d8511c8 into servo:master Jan 14, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@bors-servo bors-servo mentioned this pull request Jan 14, 2016
Closed
@frewsxcv frewsxcv mentioned this pull request Jan 14, 2016
Closed
@timvandermeij timvandermeij mentioned this pull request Jan 25, 2016
Merged
bors-servo added a commit that referenced this pull request Jan 25, 2016
@bors-servo
Auto merge of #9422 - timvandermeij:organization-cleanup, r=larsbergs…
Loading status checks…
d57ab5f 
…trom

Update ORGANIZATION.md to remove the legacy reftest harness item

The mentioned directory and file do not exist anymore. The linked issue, #5618, has been closed after PR #9293, so this item should be removed from the overview.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9422)
<!-- Reviewable:end -->
@Ms2ger Ms2ger deleted the Ms2ger:reftests branch Feb 23, 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

8 participants
@Ms2ger @SimonSapin @bors-servo @frewsxcv @jdm @KiChjang @emilio @highfive
You can’t perform that action at this time.