Skip to content

ci: Review requires each repository's Netlify previews - #28

Merged
silveltman merged 3 commits into
mainfrom
ready-netlify-previews
Oct 24, 2026
Merged

silveltman merged 3 commits into
mainfrom
ready-netlify-previews

Conversation

@silveltman

@silveltman silveltman commented Oct 24, 2026 •

Copy link
Copy Markdown
Contributor

Every website now has its own organization ruleset (`Netlify: `, 37 in total) only to require its Netlify deploy preview. Review already knows each repository's previews through `netlify-checks`, so it can require them itself:

  • Review passes only once every `netlify-checks` context of the revision is `success` (a skipped monorepo preview reports success too).
  • CloudCannon's automatic publication requires only the central Check and Review rules, so the per-site rulesets can go.
  • README: callers without Test also run Ready on Netlify's `status` events, since a preview often finishes after Check; Fulldev Sites joins CloudCannon as a content publisher with auto-merge.

Rollout: callers move to this revision (with the `status` trigger where they have no Test), then the per-site rulesets are removed.

Tests: `node --test tests/*.test.mjs` (51 pass), with Review pending while a preview is pending, failed or missing.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

silveltman and others added 3 commits October 24, 2026 11:45
Review now waits for every netlify-checks preview of the revision, so
the organization needs no Netlify rule per site, and CloudCannon's
automatic publication needs only the central Check and Review rules.
Fulldev Sites joins CloudCannon as a content publisher with auto-merge.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@silveltman
silveltman merged commit b614208 into main Oct 24, 2026
3 checks passed
@silveltman
silveltman deleted the ready-netlify-previews branch October 24, 2026 09:53
silveltman added a commit to fulldotdev/silveltman that referenced this pull request Oct 24, 2026
Moves the Ready caller to fulldotdev/.github@b614208
(fulldotdev/.github#28): Review requires this repository's Netlify
previews itself, and Ready also runs on Netlify's status events. Then
the per-site Netlify ruleset can go.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- codesmith:footer -->
---
<a
href="https://app.blacksmith.sh/fulldotdev/codesmith/silveltman/pr/42?autoLogin=true&ref=codesmith_pr_footer"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-light-v3.svg"><img
alt="View with [code]smith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3-3x.png"
width="162" height="28"></picture></a> <a
href="https://backend.blacksmith.sh/track/enable-autofix?expires=1795427664&installation_model_id=442937&pr_number=42&ref=codesmith_pr_footer&repository=fulldotdev%2Fsilveltman&return_to=https%3A%2F%2Fgithub.com%2Ffulldotdev%2Fsilveltman%2Fpull%2F42&signature=1b94ba8dd8f09087408a3e583984c803db4e8f762af9bffb696fc71178281bd1"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-light.svg"><img
alt="Autofix with [code]smith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark-3x.png"
width="190" height="28"></picture></a>
<sup>Need help on this PR? Tag <code>@codesmith-bot</code> with what you
need.</sup>

<!-- codesmith:autofix:disabled -->
<!-- /codesmith:footer -->
silveltman added a commit to fulldotdev/sitex that referenced this pull request Oct 24, 2026
Moves the Ready caller to fulldotdev/.github@b614208
(fulldotdev/.github#28): Review requires this repository's Netlify
previews itself, and Ready also runs on Netlify's status events. Then
the per-site Netlify ruleset can go.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- codesmith:footer -->
---
<a
href="https://app.blacksmith.sh/fulldotdev/codesmith/sitex/pr/40?autoLogin=true&ref=codesmith_pr_footer"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-light-v3.svg"><img
alt="View with [code]smith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3-3x.png"
width="162" height="28"></picture></a> <a
href="https://backend.blacksmith.sh/track/enable-autofix?expires=1795429816&installation_model_id=442937&pr_number=40&ref=codesmith_pr_footer&repository=fulldotdev%2Fsitex&return_to=https%3A%2F%2Fgithub.com%2Ffulldotdev%2Fsitex%2Fpull%2F40&signature=93dfb8144cb3414805d294dc13807025a3a8cb3311c664b2845eba8cc5fea309"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-light.svg"><img
alt="Autofix with [code]smith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark-3x.png"
width="190" height="28"></picture></a>
<sup>Need help on this PR? Tag <code>@codesmith-bot</code> with what you
need.</sup>

<!-- codesmith:autofix:disabled -->
<!-- /codesmith:footer -->
silveltman added a commit to fulldotdev/ui that referenced this pull request Oct 24, 2026
Moves the Ready caller to fulldotdev/.github@b614208
(fulldotdev/.github#28): Review requires this repository's Netlify
previews itself, and Ready also runs on Netlify's status events. Then
the per-site Netlify ruleset can go.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- codesmith:footer -->
---
<a
href="https://app.blacksmith.sh/fulldotdev/codesmith/ui/pr/329?autoLogin=true&ref=codesmith_pr_footer"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-light-v3.svg"><img
alt="View with [code]smith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3-3x.png"
width="162" height="28"></picture></a> <a
href="https://backend.blacksmith.sh/track/enable-autofix?expires=1795429966&installation_model_id=442937&pr_number=329&ref=codesmith_pr_footer&repository=fulldotdev%2Fui&return_to=https%3A%2F%2Fgithub.com%2Ffulldotdev%2Fui%2Fpull%2F329&signature=241e4c046f686752532812b2fecea946be5bf885995f40e6818da36affa82b2e"><picture><source
media="(prefers-color-scheme: dark)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg"><source
media="(prefers-color-scheme: light)"
srcset="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-light.svg"><img
alt="Autofix with [code]smith"
src="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark-3x.png"
width="190" height="28"></picture></a>
<sup>Need help on this PR? Tag <code>@codesmith-bot</code> with what you
need.</sup>

<!-- codesmith:autofix:disabled -->
<!-- /codesmith:footer -->
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