Skip to content

Document dynamic CI container image tag#39995

Closed
karlhorky wants to merge 1 commit intomicrosoft:mainfrom
karlhorky:docs/dynamic-container-version-ci-v2
Closed

Document dynamic CI container image tag#39995
karlhorky wants to merge 1 commit intomicrosoft:mainfrom
karlhorky:docs/dynamic-container-version-ci-v2

Conversation

@karlhorky
Copy link
Copy Markdown
Contributor

@karlhorky karlhorky commented Apr 1, 2026

Closes #39988

The CI docs hardcode the Playwright Docker image tag in workflow YAML, so it can drift from the installed Playwright version during upgrades.

This adds a short Via Containers (dynamic) subsection with:

  • a copy-pasteable GitHub Actions workflow that resolves the version from package-lock.json
  • replacement version="$(...)" lines for Yarn Berry, pnpm, and Bun

This keeps the example short while covering the lockfile direction from #37637.

Copilot AI review requested due to automatic review settings April 1, 2026 10:28
@karlhorky karlhorky force-pushed the docs/dynamic-container-version-ci-v2 branch 2 times, most recently from 785ce44 to 145f4e8 Compare April 1, 2026 10:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds documentation to the CI guide showing how to dynamically derive the Playwright Docker image tag from the project lockfile, preventing the workflow’s container image from drifting during Playwright upgrades.

Changes:

  • Add a new “Via Containers (dynamic)” subsection for JS.
  • Provide a lockparse-based script to resolve the installed @playwright/test version from common lockfiles.
  • Show a two-job GitHub Actions pattern that passes the resolved version into jobs.<job>.container.image.

@karlhorky karlhorky force-pushed the docs/dynamic-container-version-ci-v2 branch 7 times, most recently from 73ace88 to 173f509 Compare April 1, 2026 16:18
Copy link
Copy Markdown
Member

@Skn0tt Skn0tt left a comment

Choose a reason for hiding this comment

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

I like it! I Don't want to create yet another section in this doc though, do you think you can weave it into the existing JS example?

@pavelfeldman
Copy link
Copy Markdown
Member

The issue was closed

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.

[Docs]: document dynamic Playwright Docker image version in CI guide

4 participants