Skip to content

Fix Stale Version Comment on the merge-bot-task.yml Pin - #92

Merged
ptr727 merged 1 commit into
developfrom
chore/fix-merge-bot-version-comment
Aug 29, 2026
Merged

Fix Stale Version Comment on the merge-bot-task.yml Pin#92
ptr727 merged 1 commit into
developfrom
chore/fix-merge-bot-version-comment

Conversation

@ptr727

Copy link
Copy Markdown
Owner

Review on the main-promotion PR (#91) caught that the earlier bulk pin bump (a72c48c) kept
merge-bot-pull-request.yml's trailing version comment unchanged while bumping its SHA.
b823bc4... is tagged 2.0.512 upstream (confirmed via git tag --points-at against the hub),
not the 2.0.380 the stale comment still named. The other five hub-workflow uses: in this repo
use a # main promotion label instead, which never goes stale this way; this is the only one on
a version-tag convention.

🤖 Generated with Claude Code

The earlier bulk pin bump (a72c48c) replaced every hub-workflow SHA in
this repo but kept each file's existing trailing comment unchanged.
merge-bot-pull-request.yml's comment convention is a version tag
rather than the 'main promotion' label the other five uses carry, and
review on the main-promotion PR caught that the new SHA
(b823bc4...) is tagged 2.0.512 upstream, not the 2.0.380 the stale
comment still named. Confirmed via git tag --points-at against the
hub.
CopilotAI lite review requested due to automatic review settings August 29, 2026 04:16
@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

Correct merge-bot workflow version annotation

📝 Documentation🕐 Less than 5 minutes

Grey Divider

AI Description

• Corrects the reusable merge-bot workflow comment from version 2.0.380 to 2.0.512.
• Aligns the annotation with the already-pinned upstream commit.
High-Level Assessment

Correcting the annotation in place is appropriate because the pinned SHA is already valid and only its human-readable version comment is stale. Changing the pin or adopting a repository-wide annotation convention would exceed this PR’s corrective scope.

Files changed (1) +1 / -1

Documentation (1) +1 / -1
merge-bot-pull-request.ymlCorrect reusable workflow version comment+1/-1

Correct reusable workflow version comment

• Changes the trailing version annotation for the pinned merge-bot reusable workflow from 2.0.380 to 2.0.512. The pinned commit SHA and workflow behavior remain unchanged.

.github/workflows/merge-bot-pull-request.yml

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0)📘 Rule violations (0)📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can group findings by type and pick your Finding display, from Minimal to Full

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

CopilotAI 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.

🟢 Approval recommended

The change corrects a comment-only mismatch without altering workflow behavior, and the remaining note is a non-blocking maintainability suggestion.

Pull request overview

Updates the trailing version comment on the pinned hub reusable workflow reference so it matches the already-updated SHA and doesn’t misreport the upstream tag.

Changes:

  • Fixes the stale # 2.0.380 comment to # 2.0.512 for the merge-bot-task.yml pin.
File summaries
FileDescription
.github/workflows/merge-bot-pull-request.ymlUpdates the pinned workflow’s trailing version comment to reflect the correct upstream tag for the existing SHA.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread.github/workflows/merge-bot-pull-request.yml
@ptr727
ptr727 merged commit 67a0993 into developAug 29, 2026
10 checks passed
@ptr727
ptr727 deleted the chore/fix-merge-bot-version-comment branch August 29, 2026 04:19
Sign up for freeto 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

@ptr727