Skip to content
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

ci: Fix edge ci with updates #17378

Merged
merged 2 commits into from
Aug 5, 2024

Conversation

danielhjacobs
Copy link
Contributor

Supersedes #17347 and #17255.

Requesting review from @torokati44 as the one who always does the manual upgrades. See also webdriverio-community/node-edgedriver#365 (comment).

@danielhjacobs
Copy link
Contributor Author

danielhjacobs commented Aug 5, 2024

This was the result of npm remove wdio-chromedriver-service wdio-edgedriver-service wdio-geckodriver-service, a direct deletion of this:

    "optionalDependencies": {
        "chromedriver": "^127.0.1"
    },

Followed by npm install --save-dev chromedriver edgedriver geckodriver and npm install webdriverio

web/package.json Outdated Show resolved Hide resolved
@christian-bromann
Copy link

Followed by npm install --save-dev chromedriver edgedriver geckodriver

These are already dependency of @wdio/utils and don't need to be your projects dependencies 😉

@torokati44
Copy link
Member

Hmmm...

Error: Error: Failed to initilialise launcher service unknown: Error: Couldn't find plugin "chromedriver" service, neither as wdio scoped package "@wdio/chromedriver-service" nor as community package "wdio-chromedriver-service". Please make sure you have it installed!

Also, look at the typo there, @christian-bromann: 😅 initilialise

@danielhjacobs danielhjacobs force-pushed the fix-edge-ci-with-updates branch 3 times, most recently from bf343d9 to ab48b88 Compare August 5, 2024 19:11
@danielhjacobs
Copy link
Contributor Author

Tried just npm remove wdio-chromedriver-service wdio-edgedriver-service wdio-geckodriver-service and npm install webdriverio without any manual changes to the package files.

@danielhjacobs
Copy link
Contributor Author

These are already dependency of @wdio/utils and don't need to be your projects dependencies 😉

Yeah, this doesn't seem to be the case, as @torokati44 noted.

@danielhjacobs
Copy link
Contributor Author

danielhjacobs commented Aug 5, 2024

@danielhjacobs danielhjacobs force-pushed the fix-edge-ci-with-updates branch 2 times, most recently from 88d98dd to 66f5df9 Compare August 5, 2024 19:52
@christian-bromann
Copy link

You probably still have the services defined in your wdio.conf.ts/js file, you will have to remove the references there too, otherwise WebdriverIO will try to import them.

@torokati44
Copy link
Member

You probably still have the services defined in your wdio.conf.ts/js file, you will have to remove the references there too, ...

Right...: https://github.com/ruffle-rs/ruffle/blob/a00d0e94992084b3b99dba9958391e9e78c2ee8e/web/packages/selfhosted/wdio.conf.ts#L19-62

@danielhjacobs danielhjacobs force-pushed the fix-edge-ci-with-updates branch 2 times, most recently from 722cfa6 to 50fc906 Compare August 5, 2024 20:14
@christian-bromann
Copy link

Please update your package-lock.json and ensure your edgedriver dependency is on v5.6.1

@torokati44
Copy link
Member

Thanks for popping in to help, @christian-bromann, this worked!
Are you okay with merging this as is, and closing the two superseded PRs, @danielhjacobs?

@torokati44 torokati44 enabled auto-merge (rebase) August 5, 2024 21:16
@torokati44 torokati44 merged commit 57b21cf into ruffle-rs:master Aug 5, 2024
16 of 17 checks passed
@danielhjacobs danielhjacobs deleted the fix-edge-ci-with-updates branch August 5, 2024 21:28
@torokati44
Copy link
Member

Thank you for really getting to the bottom of this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants