feat(workflow-templates): add dependabot-auto-merge caller stub + properties - #7
Merged
Merged
Conversation
…perties Mirrors smartwatermelon/.github#11: replaces the self-contained dependabot-auto-merge.yml body with a caller stub pointing at the new reusable workflow in smartwatermelon/github-workflows (Part A2 of the reusable-workflows plan; see smartwatermelon/github-workflows#96). Adds the missing dependabot-auto-merge.properties.json so the workflow-templates picker actually offers this template — confirmed silently absent during Q2 planning validation. Temporarily pinned to a commit SHA rather than the dependabot-auto-merge-v1 tag, since that tag doesn't exist until github-workflows#96 merges and the tag is cut (~1 minute after merge, per this fleet's established convention). Follow-up PR needed once the tag exists. Claude-Session: https://claude.ai/code/session_0143Nf4sXqnTp2Kx3LNbMvBA
This comment has been minimized.
This comment has been minimized.
Mirrors smartwatermelon/.github#11: the reusable workflow's dependabot-auto-merge-v1.0.0 tag (and floating dependabot-auto-merge-v1) are now cut in smartwatermelon/github-workflows following #96's merge. Replace the temporary branch-tip SHA pin with the real floating tag, and document explicitly that floating (versus a stricter pin) is an accepted tradeoff for this template, plus note that the actor gate and merge-scope defaults now live entirely inside the reusable workflow, invisible from this caller stub.
|
No blocking issues found. The PR converts an inline The floating-tag pin ( VERDICT: PASS |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Mirrors smartwatermelon/.github#11. Part of Part A2/A4 of the reusable-workflows plan. Replaces this template's self-contained
dependabot-auto-merge.ymlbody with a caller stub pointing at the new reusable workflow being added in smartwatermelon/github-workflows#96. Adds the missingdependabot-auto-merge.properties.jsonso the workflow-templates picker actually offers this template (Part B's B1 deliverable per the plan — confirmed silently absent from the picker during Q2 planning validation).Actor guard — confirmed preserved
The reusable workflow (
smartwatermelon/github-workflowscommitad3f0ec, PR #96) keeps the sameif: github.actor == 'dependabot[bot]'guard, now at the reusable'sauto-mergejob level instead of this caller's — it was moved, not dropped.Blocking on tag creation
Temporarily pinned to the
github-workflowscommit SHAad3f0ec080ad5b119f2359bf42de6623c4730544(the tip of PR #96's branch at the time this PR was opened) rather than a tag, sincedependabot-auto-merge-v1doesn't exist until that PR merges and the tag is cut (~1 minute after merge, per that fleet's convention). Caveat: this SHA is real and fetchable today (it's a pushed branch tip), but if PR #96 is squash-merged, the final commit ongithub-workflowsmain will have a different SHA than this one. Follow-up PR needed here regardless — either to repoint to@dependabot-auto-merge-v1once cut, or sooner if this exact SHA stops resolving.Test plan
yamllintclean (one pre-existing-style line-length warning)dependabot.yml'sgithub-actionsecosystem entry doesn't scanworkflow-templates/, so pins there won't get automated bump PRs — noting for the record, not fixing here)@dependabot-auto-merge-v1once that tag is cut ingithub-workflowsClaude-Session: https://claude.ai/code/session_0143Nf4sXqnTp2Kx3LNbMvBA