Skip to content

Align docs-server step names across docs testing workflows#37238

Merged
pelikhan merged 2 commits into
mainfrom
copilot/align-docs-server-start-wait-steps
Jun 6, 2026
Merged

Align docs-server step names across docs testing workflows#37238
pelikhan merged 2 commits into
mainfrom
copilot/align-docs-server-start-wait-steps

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jun 6, 2026

Three docs-testing workflows perform the same docs-server bootstrap/readiness sequence but used inconsistent step names, which made cross-workflow scanning noisy. This updates the outliers to the majority naming and regenerates lock files from source specs.

  • Step name normalization (source workflow specs)

    • .github/workflows/daily-multi-device-docs-tester.md: Start documentation serverStart docs server
    • .github/workflows/visual-regression-checker.md: Wait for dev serverWait for server readiness
  • Lockfile regeneration

    • Recompiled:
      • .github/workflows/daily-multi-device-docs-tester.lock.yml
      • .github/workflows/visual-regression-checker.lock.yml
    • Result: compiled workflows now reflect the same canonical start/wait step names used across the docs-testing suite.
# before
- name: Start documentation server
- name: Wait for dev server

# after
- name: Start docs server
- name: Wait for server readiness

Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
Copilot AI changed the title [WIP] Align docs-server start/wait step names across workflows Align docs-server step names across docs testing workflows Jun 6, 2026
Copilot AI requested a review from gh-aw-bot June 6, 2026 02:42
@pelikhan pelikhan marked this pull request as ready for review June 6, 2026 03:32
Copilot AI review requested due to automatic review settings June 6, 2026 03:32
@pelikhan pelikhan merged commit 3f91460 into main Jun 6, 2026
@pelikhan pelikhan deleted the copilot/align-docs-server-start-wait-steps branch June 6, 2026 03:33
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

This PR normalizes docs-server bootstrap/readiness step names across the docs-testing workflows to reduce noise when scanning runs, and regenerates the compiled lock workflows so the rendered YAML matches the updated source specs.

Changes:

  • Renamed the visual-regression workflow step Wait for dev serverWait for server readiness.
  • Renamed the multi-device docs tester step Start documentation serverStart docs server.
  • Regenerated both corresponding .lock.yml workflows to reflect the canonical step naming.
Show a summary per file
File Description
.github/workflows/visual-regression-checker.md Renames the server readiness step for consistent naming.
.github/workflows/visual-regression-checker.lock.yml Regenerated compiled workflow to include updated step name.
.github/workflows/daily-multi-device-docs-tester.md Renames the docs server start step for consistent naming.
.github/workflows/daily-multi-device-docs-tester.lock.yml Regenerated compiled workflow to include updated step name.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 4/4 changed files
  • Comments generated: 0

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.

[step-names] Align docs-server start/wait step names across the three docs-testing workflows

4 participants