Repository navigation
Conversation
|
Code Review ✅ Approved🟡 Medium risk · The local action changes build-number allocation to use the PR head; allocator or token failures could assign an incorrect number and affect promoted artifact versioning. Replaces the build-number allocator with a repository-local action that runs the pinned v2 allocator script against the PR head, avoiding the HTTP 403 seen with synthetic merge commits, and passes the number to Build and Promote. No issues found. Verification Rules 3/3 met PRs 1 inspectedFunctional validation No results Rules 3 passed
Broader impact PRs 1 inspectedCross-PR 1 PR inspected 0 open impactsAuto-approval Not enabledFor more verification details, comment OptionsAuto-apply is off → Gitar will not commit updates to this branch. Comment with these commands to change the behavior for this request:
Was this helpful? React with 👍 / 👎 | Gitar |





Replace the allocator pinned to the closed ci-github-actions#362 draft with a repository-local action. It runs the existing released v2 allocator script against the PR head, avoiding the reproduced HTTP 403 for synthetic merge commits, then passes the number to Build and Promote.
The allocator script is pinned to the same v2 commit used during the investigation. The local wrapper preserves legacy-property migration, same-run reuse and allocation locking, and removes its temporary files. The build continues to check out and analyse the merge commit.
Validation: actionlint, YAML parsing and ShellCheck; end-to-end CI pending.