Skip to content

chore: anchor release-please to this release cycle's base - #585

Closed
Vivswan wants to merge 1 commit into
mainfrom
chore/anchor-release-boundary
Closed

Vivswan wants to merge 1 commit into
mainfrom
chore/anchor-release-boundary

Conversation

@Vivswan

@Vivswan Vivswan commented Nov 11, 2026

Copy link
Copy Markdown
Owner

What this specifies

before: "last-release-sha": "046adf3b24454f26f569850630809bcf481f8b84"
after:  "last-release-sha": "56d949aec56aaaf95ed0cd324b02f47d0f76ed97"
before: the anchor rides inside release PR #58 -> its squash merge lands the boundary on main
after:  the anchor is its own PR against main, off the release PR branch

How

  • One line in release-please-config.json moves release-please's boundary to main's tip at the time of writing.
  • The commit is the release pipeline's own, lifted unchanged from the release PR branch.

Proof

  • boundary-check fails on main with this config: NEWER than 046adf3b, exit 1. The unchanged config passes the same checker.
  • That check is skipped on this PR and runs on main, where its red fails all-green and skips post-green and release.
Technical details
  • Do not merge as it stands. It needs a matching change to the boundary policy in .github/scripts/release-pipeline/anchor.ts first.
  • Changelog effect: with the boundary at main's tip, release-please sees nothing to release, so the 3.0.0 release PR empties or closes.
  • Release PR chore(main): release 3.0.0 #58: the same commit is removed from its branch; anchor-check there expects it, and the next release-please refresh adds it back.
  • Files: release-please-config.json.

last-release-sha records the merge parent so the squash merge itself lands the next cycle's boundary on main: version tags live on packaged commits that are not on main, so release-please cannot find the boundary by tag.
Copilot AI balanced review requested due to automatic review settings November 11, 2026 18:48
@github-actions

Copy link
Copy Markdown
Contributor

File size check

0 over a hard cap (fails), 6 warning(s).

File Size Tier Cap
src/engine/diff.ts:56 11 comment lines warn 10
src/github/api-error.ts:9 11 comment lines warn 10
src/problem.ts:73 18 comment lines warn 10
src/sections/contract/errors.ts:13 11 comment lines warn 10
src/sections/contract/module.ts:55 12 comment lines warn 10
src/sections/shared/list/decl.ts:265 14 comment lines warn 10

Split the file, wrap the line, shorten or exempt the comment, or list the path in .file-size-allow.local with a # reason.

5 managed file(s) skipped; repo-platform owns them.

@github-actions

Copy link
Copy Markdown
Contributor

Member shape check

1 finding(s).

File Line Rule
test/sections/contract/validate-hook.test.ts 80 all-or-none

Add the blank lines, make the comment one line directly above the item, comment every field or none, or list the path in .member-shape-allow.local with a # reason.

4 generated file(s) skipped.

@Vivswan

Vivswan commented Nov 11, 2026

Copy link
Copy Markdown
Owner Author

Closed: the anchor commit is back on the release PR branch (#58), where the release pipeline lands it.

@Vivswan Vivswan closed this Nov 11, 2026
@Vivswan
Vivswan deleted the chore/anchor-release-boundary branch November 11, 2026 18:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new release anchor is rejected by the unchanged boundary policy, causing main to fail boundary-check and blocking publishing and releases despite green PR checks.

1 open issue
What changed in this PR

This PR separates the release boundary update from release PR #58, moving release-please's history cutoff to the main tip recorded when this PR was prepared.

Changes:

  • Updates last-release-sha from 046adf3b... to 56d949ae....
  • Leaves the release pipeline and boundary validation policy unchanged.
File Description
release-please-config.json Advances the commit boundary used by release-please.

Review effort: Balanced

"bump-patch-for-minor-pre-major": true,
"include-component-in-tag": false,
"last-release-sha": "046adf3b24454f26f569850630809bcf481f8b84",
"last-release-sha": "56d949aec56aaaf95ed0cd324b02f47d0f76ed97",
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.

2 participants