Skip to content
This repository was archived by the owner on Dec 16, 2024. It is now read-only.

Create error summary logs when running wptrunner-based tests - #310

Merged
bors-servo merged 1 commit into
servo:masterfrom
jgraham:wpt_errorsummary
Apr 20, 2016
Merged

bors-servo merged 1 commit into
servo:masterfrom
jgraham:wpt_errorsummary

Conversation

@jgraham

@jgraham jgraham commented Apr 8, 2016

Copy link
Copy Markdown
Contributor

This change is Reviewable

@aneeshusa

Copy link
Copy Markdown
Contributor

You'll need to update the logfiles parameter as well for each ShellCommand so Buildbot knows to capture the new log files. Other than that, LGTM.

@Ms2ger

Ms2ger commented Apr 20, 2016

Copy link
Copy Markdown
Contributor

@jgraham ping

@jgraham

jgraham commented Apr 20, 2016

Copy link
Copy Markdown
Contributor Author

Done. Sorry missed this review in the first instance.

@Ms2ger

Ms2ger commented Apr 20, 2016

Copy link
Copy Markdown
Contributor

@bors-servo r+

@bors-servo

Copy link
Copy Markdown
Contributor

📌 Commit 5a7b867 has been approved by Ms2ger

@bors-servo

Copy link
Copy Markdown
Contributor

⌛ Testing commit 5a7b867 with merge 3a50c3d...

bors-servo pushed a commit that referenced this pull request Apr 20, 2016
Create error summary logs when running wptrunner-based tests

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/310)
<!-- Reviewable:end -->
@bors-servo

Copy link
Copy Markdown
Contributor

☀️ Test successful - travis

@bors-servo
bors-servo merged commit 5a7b867 into servo:master Apr 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants