Skip to content

fix(e2e): remove unnecessary workaround for tests-playwright version - #4683

Closed
axel7083 wants to merge 1 commit into
containers:mainfrom
axel7083:fix/remove-tests-playwright-workaround
Closed

fix(e2e): remove unnecessary workaround for tests-playwright version#4683
axel7083 wants to merge 1 commit into
containers:mainfrom
axel7083:fix/remove-tests-playwright-workaround

Conversation

@axel7083

Copy link
Copy Markdown
Contributor

Description

The step was dynamically replacing the @podman-desktop/tests-playwright version with the latest @next release before running pnpm install. With pnpm v11 minimumReleaseAge defaulting to 1440 minutes, this breaks CI when a @next version was published within the last 24 hours.

The workaround referenced podman-desktop/extension-bootc#712 and is no longer needed.

Related issues

Required for #4682

The step was dynamically replacing the @podman-desktop/tests-playwright
version with the latest @next release before running pnpm install.
With pnpm v11 minimumReleaseAge defaulting to 1440 minutes, this breaks
CI when a @next version was published within the last 24 hours.

The workaround referenced podman-desktop/extension-bootc#712
and is no longer needed.

Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

@bmahabirbu bmahabirbu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

seems ci is failing wondering if its just flaky?

@axel7083

Copy link
Copy Markdown
Contributor Author

Closing in favour of adding minimumReleaseAgeExclude: ['@podman-desktop/tests-playwright'] to pnpm-workspace.yaml in the pnpm v11 migration PR, which is the correct long-term fix per QE decision.

@axel7083 axel7083 closed this Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants