Skip to content

Dependency updates: signed version commit, so an automatic update can merge on its own (version 0.4.7) - #53

Merged
EaZyCheaZ merged 2 commits into
devfrom
p1-dependency-updates-signed
Oct 25, 2026
Merged

EaZyCheaZ merged 2 commits into
devfrom
p1-dependency-updates-signed

Conversation

@EaZyCheaZ

@EaZyCheaZ EaZyCheaZ commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Task 4f follow-up.

#48 (web app packages, patch and minor) had its version commit and a passing dispatched build, but GitHub refused the merge: "the base branch policy prohibits the merge". The dev ruleset has require_extra_approval_for_unattributed_changes on (it was on before this task, and the branch rules stay as they are), and the version commit the workflow pushed with git is unsigned (github-actions[bot], unverified).

The update script now makes that commit through GitHub's API (blobs, tree, commit, ref) with the workflow's token, which GitHub signs. The local commit is still made first and the API's tree has to match it exactly, so the content is unchanged. Only files that differ from the closest parent are sent (dev, when dev is merged in). If GitHub refuses, the update is held as before.

Second commit: the update job now waits for the build it started (up to 45 minutes) and then merges or holds, then goes on to the next update. GitHub started no run when a dispatched build finished (the workflow_run trigger only fired for dev's push build), nor when auto-merge turned on by the workflow merged, and the hourly schedule has not fired once since #44. The workflow_run trigger is removed; job timeout 120 minutes.

Checked: the API path against a throwaway branch (tree identical, branch moved, branch deleted); self-test 17 of 17. Whether GitHub then treats the signed commit as attributed is shown by the next automatic update after this merges.

Version 0.4.7, heading only.

…t is signed and can merge without an extra approval (version 0.4.7)
…ge or hold, instead of relying on another run
@EaZyCheaZ
EaZyCheaZ merged commit a722de1 into dev Oct 25, 2026
23 checks passed
@EaZyCheaZ
EaZyCheaZ deleted the p1-dependency-updates-signed branch October 25, 2026 10:42
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