-
Notifications
You must be signed in to change notification settings - Fork 29
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
test sauce labs tests #214
Conversation
I've had to directly commit the workflow stuff here because the tests wouldn't run anymore because of the higher node version required. At least the tests run in chrome again now which they didn't before. |
Maybe I could try to use a patch (eg https://www.npmjs.com/package/patch-package) and get the Saucelabs tunnel in a working state. I'll have a look when I get home on both these PRs. The delay on both is due to the broken remote tests... |
@wilsonge so I forked the karma-sauce-launcher and applied the PR and the remote tests are green again. Let me know if that's ok as a temp solution, till the upstream repo merges the karma-runner/karma-sauce-launcher#278 |
karma-runner/karma-sauce-launcher#275 (comment) Are we able to stay with the original and use this workaround? If not your fork is fine. We just need to document somewhere for the future in case they are able to magically deploy things. |
@wilsonge applied the override, so no more forks. Merge it and let's see if it actually works |
Looks good. Am I merging this one or you going to push to #215 this new part? |
Merge this one, I have to review the other again (it's been a while and some pkgs might need another update) |
DO NOT MERGE THIS!!!!!