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

Do not override :wait for non-Playwright drivers #89

Merged
merged 1 commit into from
Dec 21, 2024

Conversation

arzezak
Copy link
Contributor

@arzezak arzezak commented Dec 18, 2024

Do not override :wait when the current driver is not Playwright. This was causing our suite to fail just by requiring
capybara-playwright-driver.

Do not override `:wait` when the current driver is not Playwright. This
was causing our suite to fail just by requiring
`capybara-playwright-driver`.

Co-authored-by: mauro-oto <mauro.otonelli@gmail.com>
@ashkulz
Copy link

ashkulz commented Dec 19, 2024

We too hit this, worked it around by using require: false in the Gemfile and instead doing it inside the Capybara.register_driver block as we're slowly migrating our existing test suite to Playwright.

Copy link
Owner

@YusukeIwaki YusukeIwaki left a comment

Choose a reason for hiding this comment

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

It looks good :)

@YusukeIwaki YusukeIwaki merged commit a3fd1a5 into YusukeIwaki:main Dec 21, 2024
11 checks passed
@YusukeIwaki
Copy link
Owner

YusukeIwaki commented Dec 21, 2024

@arzezak @ashkulz Thank you for your contribution. This fix is included into v0.5.4, and now it is available.
Please try it. Thank you! :)

@arzezak arzezak deleted the override-playwright-wait-only branch December 21, 2024 14:16
@arzezak
Copy link
Contributor Author

arzezak commented Dec 21, 2024

Thank you very much, @YusukeIwaki, for your efforts in working on this.

@ashkulz
Copy link

ashkulz commented Dec 23, 2024

@YusukeIwaki any particular reason 0.5.3 was yanked from RubyGems?

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