Skip to content

Test: PR media check (do not merge) - #21

Closed
philmillman wants to merge 3 commits into
mainfrom
test-media-check-pr
Closed

philmillman wants to merge 3 commits into
mainfrom
test-media-check-pr

Conversation

@philmillman

Copy link
Copy Markdown
Owner

Verifies the CI changes from a8faf8c on a pull request:

  1. This commit touches elektron-machinedrum/sm.yaml, which has a hotlinked image. The validate step should run in strict mode and check only this file's media, and the build step should run with the media check off.
  2. A second commit will point that image at a 404, which strict mode should fail.

Close without merging once both runs are checked.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://test-media-check-pr-altstrument.philmillman.workers.dev (commit fdb9077)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://8aad111c-altstrument.philmillman.workers.dev fdb9077 2026-11-06T16:36:08.402Z Visit the dashboard ↗
  • Build: Failed ❌

View logs ↗
95dd88d 2026-11-06T16:31:31.238Z View logs ↗

@philmillman

Copy link
Copy Markdown
Owner Author

Verified: strict PR check fails on the 404 with an annotation; Cloudflare preview build only warns. Previews block landed on main as 3f40367.

@philmillman philmillman closed this Nov 6, 2026
@philmillman
philmillman deleted the test-media-check-pr branch November 6, 2026 16:41
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