-
-
Notifications
You must be signed in to change notification settings - Fork 427
feat: add Nightwatch.js as test framework option #572
Conversation
Do you mind resolving the conflicts so we can merge it? |
No problem @atinux I'll check it this evening or tomorrow and rebase this branch with master resolving the conflict. |
aaafa12
to
4cf9466
Compare
4cf9466
to
7b7f883
Compare
renovate.json
Outdated
@@ -57,6 +57,10 @@ | |||
{ | |||
"packagePatterns": ["wdio", "webdriver"], | |||
"groupName": "webdriverio packages" | |||
}, | |||
{ | |||
"packagePatterns": ["nightwatch"], |
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.
Should chromedriver
be included ?
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.
yes, it needs at least one driver to run tests, chromedriver
is the most common one.
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.
@clarkdo sorry I replied too fast here! I've just added chromedriver in renovate.json, I forgot to add it before!
@astagi I've updated snapshot job, can you please merge latest master and try ? |
7b7f883
to
b586810
Compare
b586810
to
1e790b4
Compare
Thanks! @astagi |
This PR adds Nightwatch.js as test framework option 🦉