-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Re-enable some browser-testing CI #322
Comments
Great research, any of these is ok for me as long as it works. 👍 |
By the way, it seems that you didn't actually removed sauceLabs : I see it running on v3 commits. |
You can write to: [email protected] I have tried adding these to travis, though travis is currently crashing. It says I don't have enough permissions on https://travis-ci.org/ and once I have moved to travis-ci.com, it does not work at all. |
I have given you admin access to this repo, maybe this repo will appear on your travis-ci account |
Thank you for supporting this @drzraf ! |
Actually Saucelabs is for free for opensource projects (https://saucelabs.com/open-sauce); If you are looking for alternatives let's try to see if they offer similar plans |
@evilaliv3 good to see you back. @drzraf is migrating flow.js to v3 version with some breaking changes. He is mostly redoing each callback and event in promise based approach |
@evilaliv3 TestingLabs is free for FLOSS too (I added you there too btw). If it's "just" that CI wouldn't run for PR done against the v3 branch then we may not need to switch (unless you still find other compelling argument in favor). If we decide to switch, the PR is almost ready. @AidasK : I'm not used to GitHub CI settings/travis integration but I'll have a look. |
I'm on it, let me check and i will be back to you |
@drzraf : Actually saucelabs accepts all the builds we make and we don't have conditional code in relation to the branch name. I suspect the bug could lie in the addition of these lines:
I cannot retest now but i would suggest to remove those and give it an other try |
@evilaliv3 : You're right.
The two possible path are: If SauceLabs is simply fixed (#327): Do you still think OP (karma-plugin /support) is a compelling arguments for switching or not? Are there others? |
@AidasK, could you have a look at travis-ci in order to enable it for v3? |
I sent their support a message. BTW when enabling Travis-CI for "All repositories", I only see "fusty-flow.js" and "ngx-flow". |
Great! It must be an issue at Travis side. fusty-flow.js is a fallback for flow.js for old browsers such as IE8. It’s no longer relevant. |
I was able to do without Travis at all (Only saucelabs and codeclimate). Will do a PR soon. |
One issue with SauceLabs is that the karma plugin apparently got outdated (because protocol evolves, etc...)
Now there is also this karma-webdriver-launcher plugins which aims to be more standard via webdriver (which wap them all).
It's said to support SauceLabs and even advised by lambdatest.
Even more interesting (from the above pricing), TestingBot says it's supported by webdrive.io.
@AidasK ?
The text was updated successfully, but these errors were encountered: