Skip to content

Conversation

@stee-re
Copy link

@stee-re stee-re commented Apr 23, 2025

playwright shipped with open-wc is old and is not supported with Ubuntu 24.04

This is a small fix to override the stock version of playwright shipped with open-wc to a newer version which is supported on Ubuntu 24.04.
This fix was originally applied on OmicronEnergyOSS/open-scd-core - so its tried and tested.

@ca-d
Copy link

ca-d commented Apr 23, 2025

Thank you for this work! Can you see why visual regression tests would be failing here while they are still green on the parent commit?

@ca-d
Copy link

ca-d commented Apr 23, 2025

@stee-re : The "update screenshots" workflow in this repo can only be triggered manually, and it seems to have been disabled on your fork. Hence I do not know how to update the screenshots on this so that it might pass review.

@ca-d ca-d marked this pull request as draft April 23, 2025 10:36
@stee-re stee-re force-pushed the ci_fix-playwright-version branch from b39122c to bfc2ce2 Compare April 23, 2025 12:25
playwright shipped with open-wc is old and is not supported with Ubuntu
24.04
@stee-re stee-re force-pushed the ci_fix-playwright-version branch 2 times, most recently from e9786db to ebc20c5 Compare April 24, 2025 12:02
@stee-re
Copy link
Author

stee-re commented Apr 24, 2025

@ca-d I experienced the exact same test errors here when playwright worked again - namely the "displays a menu on button click" failing on webkit in German. Added the same fixes here as I did in the other repo to fix this here.

@stee-re stee-re force-pushed the ci_fix-playwright-version branch 3 times, most recently from 4909922 to 1979e8f Compare April 24, 2025 12:39
@stee-re stee-re force-pushed the ci_fix-playwright-version branch from 1979e8f to 596d6bf Compare April 24, 2025 15:13
@stee-re stee-re marked this pull request as ready for review April 24, 2025 15:51
@ca-d ca-d requested a review from JakobVogelsang June 8, 2025 15:48
@stee-re stee-re closed this Jul 1, 2025
@stee-re stee-re deleted the ci_fix-playwright-version branch July 1, 2025 05:47
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.

2 participants