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

update port chromedriver #500

Merged

Conversation

KarinG
Copy link
Contributor

@KarinG KarinG commented Sep 2, 2024

Overview

Chromedriver uses a random port by default now.

@civibot civibot bot added the 4.0.x label Sep 2, 2024
@KarinG
Copy link
Contributor Author

KarinG commented Sep 4, 2024

Not meant to make D11 pass - but meant to solve test new failures on D10. For WFC we actually also ended up downgrading Chromedriver [similar to how Drupal core handled this issue when Olivero FunctionalJS tests started failing for D10].

@jackrabbithanna
Copy link
Collaborator

Thanks Karin, going to merge. We're working on making the D11 tests pass and those failures are unrelated.

@jackrabbithanna jackrabbithanna merged commit c5b0980 into eileenmcnaughton:4.0.x Sep 4, 2024
2 of 3 checks passed
puresyntax71 pushed a commit to puresyntax71/civicrm_entity that referenced this pull request Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants