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

feat: migrate to zip.js #316

Merged
merged 2 commits into from
May 28, 2024

Conversation

johannesrue
Copy link
Contributor

Copy link
Contributor

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for raising the PR. It seems like the e2e tests are failing for this one 🤔

@johannesrue
Copy link
Contributor Author

Hm i don't have access to a mac os system to reproduce the issue. I'm developing on linux and the test for that looks fine. Same applies when i run it locally. So i don't know whats wrong there?!
ms_edge_driver_install

src/install.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks a lot!

👍

@christian-bromann christian-bromann merged commit 907c6ba into webdriverio-community:main May 28, 2024
5 checks passed
@wdio-bot
Copy link
Contributor

Hey johannesrue 👋

Thank you for your contribution to WebdriverIO! Your pull request has been marked as an "Expensable" contribution. We've sent you an email with further instructions on how to claim your expenses from our development fund. Please make sure to check your spam folder as well. If you have any questions, feel free to reach out to us at [email protected] or in the contributing channel on Discord.

We are looking forward to more contributions from you in the future 🙌

Have a nice day,
The WebdriverIO Team 🤖

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

Successfully merging this pull request may close these issues.

3 participants