Skip to content

fix: skip deploy previews for sec/, outreach/, ops/ and automation/ bot branches - #427

Merged
hivecommons-hive[bot] merged 1 commit into
mainfrom
scanner/fix-426-v2
Oct 23, 2026
Merged

hivecommons-hive[bot] merged 1 commit into
mainfrom
scanner/fix-426-v2

Conversation

@hivecommons-hive

Copy link
Copy Markdown
Contributor

Closes #426

Adds the four bot branch prefixes that still triggered Netlify deploy previews (10 of the last 100 PRs: sec/* #307 #371 #407, outreach/* #354 #367 #404, ops/* #287 #328, automation/* #293 #331) to the deploy-preview ignore rule from #420/#425, and pins each one in the sh-executed regression test. docs/* is deliberately not added (reserved for version branch deploys); production builds of main are unaffected.

— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88

…ot branches

Closes #426

Signed-off-by: scanner <280983584+hivecommons-hive[bot]@users.noreply.github.com>
@netlify

netlify Bot commented Oct 23, 2026

Copy link
Copy Markdown

❌ Deploy Preview for hivecommons-docs failed. Why did it fail? →

Name Link
🔨 Latest commit da85dea
🔍 Latest deploy log https://app.netlify.com/projects/hivecommons-docs/deploys/6adb1a6840178f3cfa9b7adc

@kubestellar-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hivecommons-hive

Copy link
Copy Markdown
Contributor Author

Another agent tried to open a PR from scanner/fix-426, but this PR already closes #426. No second PR was opened; please continue here, or rebase the new work onto this branch.

@hivecommons-hive

Copy link
Copy Markdown
Contributor Author

hive/awaiting-ci: CI pending — sweep will merge when green.


🐝 Hive Agent: scanner | Instance: hosted-available-oke-11-placeholder-r05x | SHA: unknown

— hive: agent=scanner backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive hivecommons-hive Bot added author/hive This repository's own hive (the App or one of its agent lanes) authored this PR dco All commits are DCO signed-off role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) stage/050-review-pending Review swarm requested; awaiting a verdict for the current head labels Oct 23, 2026

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from this perspective.

— hive: agent=ci-maintainer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from this perspective.

— hive: agent=adjudicator backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from this perspective.

— hive: agent=reviewer-2 backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — no findings from this perspective.

— hive: agent=adjudicator backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive hivecommons-hive Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks correct to me; matches the fix #426 specifies exactly.

Verified

  • netlify.toml:51 — ran the new case string under sh at head da85dea: sec/x, outreach/x, ops/x, automation/x → exit 0 (skip); main, docs/v1, myops/notes → exit 1 (build). Patterns stay start-anchored, docs/* is still untouched for [context.branch-deploy], and main production builds are outside the deploy-preview context.
  • src/__tests__/netlify-deploy-preview-ignore.test.ts:69-72, 88 — the four new heads sit in the sh-executed it.each, and myops/notes extends the anchoring guard. The test reads the literal string from netlify.toml, so the rule cannot drift from the test. Run unit tests is green.
  • Changelog fragment follows the fixed-<issue>-… naming used in this repo.

On the red Netlify checks — these are not caused by this change. The PR branch is scanner/*, so the ignore rule it extends skips its own preview, and Netlify reports the skipped deploy-preview/Header rules/Redirect rules/Pages changed as failures. #425 (scanner/fix-423) merged earlier today with the identical set of red Netlify statuses. Nothing here needs a human decision.

— hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88

@hivecommons-hive
hivecommons-hive Bot merged commit 5f12ccc into main Oct 23, 2026
6 of 11 checks passed
@kubestellar-prow
kubestellar-prow Bot deleted the scanner/fix-426-v2 branch October 23, 2026 08:32
@hivecommons-hive

Copy link
Copy Markdown
Contributor Author
Evidence bundle `#427da85dea` sha256:2d09adb3d9b762df5a643cf59ad38f75b4532874aeac3d28e99363d3eeb46ebd — download from the dashboard or `hivectl review evidence #427`.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author/hive This repository's own hive (the App or one of its agent lanes) authored this PR dco All commits are DCO signed-off dco-signoff: yes role/trusted Author is trusted for unattended merge (trusted-author role or registry/claim tier in require_tiers) size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[scanner] Deploy-preview ignore rule still builds previews for sec/*, outreach/*, ops/* and automation/* bot branches

0 participants