Add badges, switch to concurrent full test runs, including Safari. #26
Conversation
ryan-roemer
force-pushed
the
chore-ci-badges
branch
from
38f5065 to
f2fdc97
Compare
ryan-roemer
changed the title
ryan-roemer
force-pushed
the
chore-ci-badges
branch
from
f2fdc97 to
5dd8be6
Compare
| { "TEST_FUNC_PORT": 3070, "ROWDY_SETTINGS":"sauceLabs.safari_7_OS_X_10_9_Desktop" }, | ||
| { "TEST_FUNC_PORT": 3080, "ROWDY_SETTINGS":"sauceLabs.chrome_latest_Windows_2012_R2_Desktop" }, | ||
| { "TEST_FUNC_PORT": 3090, "ROWDY_SETTINGS":"sauceLabs.firefox_latest_Windows_2012_R2_Desktop" }, | ||
| { "TEST_FUNC_PORT": 3100, "ROWDY_SETTINGS":"local.firefox" } ]' |
There was a problem hiding this comment.
Might want to wait for this feature so you don't get a confusing browser matrix result if one fails: FormidableLabs/builder#64
There was a problem hiding this comment.
@exogen -- This replaces the same existing matrix.
The badge is a "nice to have". But having any browser + OS combination failure is something that will not be left unattended -- there are no "acceptable" failures for this project.
There was a problem hiding this comment.
And in case it's not clear: This line is replacing
There was a problem hiding this comment.
Hm, wouldn't have magellan needed the --bail_fast option before for it to be equivalent then? Either way, it seems to me that you'd still like to know if a change just broke one browser or every browser. But it doesn't matter too much.
There was a problem hiding this comment.
Future work for "no bail" option captured with #27
|
Nice! I'm migrating the huge script loader build matrix to use btw, size badges for files on npmcdn are now supported, like so: |
Add badges, switch to concurrent full test runs, including Safari.
builderfor various parallel helperssafari_7_OS_X_10_9_Desktopon Sauce fails a lot. #5/cc @exogen