Skip to content

Renames component fixture ci workflow to "component-fixtures.yml" / Component Fixture Screenshots & Tests#315218

Merged
hediet merged 1 commit into
mainfrom
hediet/b/fancy-lamprey
May 8, 2026
Merged

Renames component fixture ci workflow to "component-fixtures.yml" / Component Fixture Screenshots & Tests#315218
hediet merged 1 commit into
mainfrom
hediet/b/fancy-lamprey

Conversation

@hediet
Copy link
Copy Markdown
Member

@hediet hediet commented May 8, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 8, 2026 09:33
@hediet hediet enabled auto-merge (rebase) May 8, 2026 09:33
@hediet hediet self-assigned this May 8, 2026
@hediet hediet disabled auto-merge May 8, 2026 09:35
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 consolidates component fixture screenshot capture and Playwright-based fixture tests under a newly introduced component-fixtures.yml workflow, while updating naming and documentation to reflect the new workflow identity.

Changes:

  • Introduces a new component-fixtures.yml workflow for component fixture screenshots + tests.
  • Updates the existing screenshot-test.yml workflow’s naming/concurrency and adds Playwright fixture test execution steps.
  • Removes the standalone component-fixture-tests.yml workflow and updates skill documentation to reference the new workflow file.
Show a summary per file
File Description
.github/workflows/screenshot-test.yml Renames/rebrands the workflow and adds Playwright fixture test steps.
.github/workflows/component-fixtures.yml Adds a new workflow file that runs component fixture screenshots and Playwright tests.
.github/workflows/component-fixture-tests.yml Deletes the old standalone component fixture Playwright test workflow.
.github/skills/chat-customizations-editor/SKILL.md Updates documentation to reference component-fixtures.yml as the CI gate.

Copilot's findings

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

Comment on lines +1 to 3
name: Component Fixture Screenshots & Tests

on:
Comment on lines +85 to +91
- name: Install Playwright Chromium
run: npx playwright install chromium

- name: Install Playwright test dependencies
run: npm ci
working-directory: test/componentFixtures/playwright

@hediet hediet enabled auto-merge (squash) May 8, 2026 09:48
@hediet hediet force-pushed the hediet/b/fancy-lamprey branch from 90ff3df to fe6d51e Compare May 8, 2026 09:49
@hediet hediet merged commit 66bdcf9 into main May 8, 2026
25 of 26 checks passed
@hediet hediet deleted the hediet/b/fancy-lamprey branch May 8, 2026 10:47
@vs-code-engineering vs-code-engineering Bot added this to the 1.120.0 milestone May 8, 2026
lszomoru pushed a commit that referenced this pull request May 8, 2026
…omponent Fixture Screenshots & Tests (#315218)

Renames component fixture ci workflow to "component-fixtures.yml" / Component Fixture Screenshots & Tests
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