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
Fix headless endpoint #3876
Fix headless endpoint #3876
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just check-check-double-check, how will this impact the saucelabs
module
Codecov Report
@@ Coverage Diff @@
## master #3876 +/- ##
==========================================
+ Coverage 98.57% 98.59% +0.02%
==========================================
Files 153 153
Lines 3434 3484 +50
Branches 750 771 +21
==========================================
+ Hits 3385 3435 +50
Misses 44 44
Partials 5 5
Continue to review full report at Codecov.
|
This doesn't affect the saucelabs module at all. The saucelabs module has the right configurations: https://github.com/saucelabs/node-saucelabs/blob/master/src/utils.js#L36 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* fix headless endpoint * fix unit test * fix unit test
* fix headless endpoint * fix unit test * fix unit test
Proposed changes
We now have the final endpoint for Sauce headless, let's update all appearances.
Types of changes
Checklist
Further comments
Reviewers: @webdriverio/technical-committee