-
Notifications
You must be signed in to change notification settings - Fork 30
(WIP) Upgrade wdio-sync to support node@12 #156
base: master
Are you sure you want to change the base?
Conversation
fiber@4, a dependency of wdio-sync, is required. See laverdet/node-fibers#409
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.
this package version doesn't exist
Ref #155 |
Sorry that I didn't test it properly before making the pull request. Will push more chance. Hold on. |
Now the unit tests fail with |
Downgraded @wdio/sync to 5.8.0 the first version introduced fibers@4. Hope that would bring the gap between [email protected] a little closer... Can you take a look and fix the failing unit tests? Thanks. |
@renfeng |
Thanks for pointing it out! Then we'll have to wait for the next release version of wdio-sync. |
I would recommend to upgrade to WebdriverIO v5 |
@christian-bromann so v4 is not maintained anymore and there will be no new releases? I did not find any information on this matter. |
There will, but people haven't had the chance to make that release. Anyway I urgently recommend to move to v5. |
fiber@4, a dependency of wdio-sync, is required. See laverdet/node-fibers#409