-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix w3c false issues #10
Conversation
This is another fix at probably the correct level for https://www.drupal.org/project/drupal/issues/3458751#comment-15667563 |
Closing and reopening in order to run the tests (see #11) |
@@ -19,7 +19,23 @@ services: | |||
- ddev-global-cache:/mnt/ddev-global-cache | |||
chrome: | |||
container_name: ddev-${DDEV_SITENAME}-chrome | |||
image: selenium/standalone-chrome:latest | |||
image: selenium/standalone-chromium:latest |
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.
This change to the chrome image allows ddev to work on arm laptops. selenium/standalone-chrome:latest is not available on arm... whereas selenium/standalone-chrome:4.8 is.... go figure.
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.
I suspect you might've already seen it, but there's this fairly recent post about multi-arch images. (Doesn't explain why it worked with 4.8, but it does speak as to why generally the chrome
image doesn't support arm.)
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.
I've tested this patch on multiple of our sites with large behat test suites and it works fine. Thx everyone 👍🏼
No description provided.