Skip to content

Conversation

@danbarr
Copy link
Collaborator

@danbarr danbarr commented Jan 27, 2026

Description

Fixes the highlighed lines in the Playwright code examples to account for the newly added args.

Type of change

  • Bug fix (typo, broken link, etc.)

Related issues/PRs

Copilot correctly caught these in #471 but suppressed due to low confidence, so I missed them.

Submitter checklist

Content and formatting

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Reviewer checklist

Content

  • I have reviewed the content for technical accuracy
  • I have reviewed the content for spelling, grammar, and style

Signed-off-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
Copilot AI review requested due to automatic review settings January 27, 2026 21:19
@vercel
Copy link

vercel bot commented Jan 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
docs-website Ready Ready Preview, Comment Jan 27, 2026 9:19pm

Request Review

Copy link
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

This PR fixes the highlighted line numbers in Playwright code examples that were incorrectly adjusted after PR #471 added the --host 0.0.0.0 argument to the Kubernetes manifests.

Changes:

  • Updated highlighted line numbers in the playwright-restricted.yaml example from {16-17} to {18-19} to correctly highlight the --allowed-origins argument
  • Updated highlighted line numbers in the playwright-with-volume.yaml example from {16-17,22-25,28-31} to {18-19,24-27,30-33} to correctly highlight the output directory args, volumes section, and volumeMounts section

@danbarr danbarr merged commit 0bf97c8 into main Jan 27, 2026
10 of 12 checks passed
@danbarr danbarr deleted the fix-playwright-line-highlights branch January 27, 2026 21:23
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.

3 participants