Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upreadme: add build status badges #10482
Conversation
|
Awesome - thanks Matthias! |
|
Thank you for submitting this and linking to the demo! I'm not convinced that this is very useful for Servo, however, since we have automation that prevents code from merging if any part of the build or test process fails on any supported platform. The fact that the appveyor builds are failing right now is a problem with the appveyor setup, rather than an indication that Servo can't be built on Windows in its current state. |
|
Hmm. Besides it might be good for users to see that the project cares about stability and that the master branch is in buildable state. I can hide the appveyor badge if you want. |
|
Yeah, the travis situation was an unfortunate mismatch between what runs in |
|
There's always options:
|
|
Reassigning to @jdm. |
|
Well, given how often recently we've had failures that are legitimate and indicative of real problems, I'm going to go ahead and merge this. Hopefully they'll stop being necessary soon :) @bors-servo: r+ |
|
|
readme: add build status badges Preview can be seen here: https://github.com/matthiaskrgr/servo/tree/readme#the-servo-parallel-browser-engine-project <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10482) <!-- Reviewable:end -->
|
|
|
@bors-servo retry |
|
|
|
|
matthiaskrgr commentedApr 8, 2016
Preview can be seen here:
https://github.com/matthiaskrgr/servo/tree/readme#the-servo-parallel-browser-engine-project
This change is