Skip to content

fix(ci): restore required visual check name - #2358

Merged
dancormier merged 1 commit into
mainfrom
dcormier/restore-visual-no-op
Aug 3, 2026
Merged

fix(ci): restore required visual check name#2358
dancormier merged 1 commit into
mainfrom
dcormier/restore-visual-no-op

Conversation

@dancormier

Copy link
Copy Markdown
Contributor

Summary

  • restore the previously verified successful no-op behavior for unrelated changes
  • preserve the required Visual Regression Tests / Visual Regression Tests check identity
  • skip checkout, LFS, dependency installation, Docker, and visual execution at the step level
  • correct the README description

Why

Skipping the reusable job at the job level caused GitHub to report Visual Regression Tests / inputs.command_description. The required Visual Regression Tests / Visual Regression Tests context was never created, leaving eligible PRs blocked indefinitely.

Testing

  • actionlint
  • git diff --check
  • verified this workflow is byte-for-byte identical to the implementation exercised successfully in Actions run 30842253081
  • independent workflow review

@netlify

netlify Bot commented Aug 3, 2026

Copy link
Copy Markdown

Deploy Preview for stacks ready!

Name Link
🔨 Latest commit 6010842
🔍 Latest deploy log https://app.netlify.com/projects/stacks/deploys/6a70fa9457421c0008996f1b
😎 Deploy Preview https://deploy-preview-2358--stacks.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Aug 3, 2026

Copy link
Copy Markdown

Deploy Preview for stackoverflow-email ready!

Name Link
🔨 Latest commit 6010842
🔍 Latest deploy log https://app.netlify.com/projects/stackoverflow-email/deploys/6a70fa958bc4360008885d10
😎 Deploy Preview https://deploy-preview-2358--stackoverflow-email.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@changeset-bot

changeset-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6010842

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dancormier
dancormier marked this pull request as ready for review August 3, 2026 20:32
@dancormier
dancormier requested a review from a team as a code owner August 3, 2026 20:32
@dancormier
dancormier merged commit 725166e into main Aug 3, 2026
20 of 23 checks passed
@dancormier
dancormier deleted the dcormier/restore-visual-no-op branch August 3, 2026 20:32
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.

1 participant