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

Build: fix playwright unsynced version in CI #20778

Merged
merged 2 commits into from
Jan 25, 2023

Conversation

ndelangen
Copy link
Member

@ndelangen ndelangen commented Jan 25, 2023

What I did

  • add a resolution for playwright to ensure that all sandboxes use the same version as the docker image

This will prevent future failures of our CI caused by mismatching version of playwright

@ndelangen ndelangen changed the title add a resolution for playwright to ensure that all sandboxes use the same version as the docker image Build: fix playwright unsynced version in CI Jan 25, 2023
@ndelangen ndelangen self-assigned this Jan 25, 2023
@ndelangen ndelangen added the build Internal-facing build tooling & test updates label Jan 25, 2023
@ndelangen ndelangen requested a review from yannbf January 25, 2023 09:56
scripts/utils/yarn.ts Outdated Show resolved Hide resolved
scripts/utils/yarn.ts Outdated Show resolved Hide resolved
@ndelangen ndelangen requested a review from yannbf January 25, 2023 10:07
Copy link
Member

@yannbf yannbf left a comment

Choose a reason for hiding this comment

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

You're awesome @ndelangen

packageJson.resolutions = {
...storybookVersions,
'enhanced-resolve': '~5.10.0', // TODO, remove this
playwright: '1.30.0', // this is for our CI test, ensure we use the same version as docker image, it should match version specified in `./code/package.json` and `.circleci/config.yml`
Copy link
Member

Choose a reason for hiding this comment

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

Oh, maybe we can even import package json version and use it here? so we don't have to maintain two places and once one is updated, this is also updated?

@ndelangen ndelangen merged commit 907b4f5 into next Jan 25, 2023
@ndelangen ndelangen deleted the norbert/ensure-playwright-version-sync-in-sandboxes branch January 25, 2023 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Internal-facing build tooling & test updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants