Skip to content

Change commit message topic to 'container image digests'#127614

Merged
richlander merged 1 commit intomainfrom
richlander-patch-1
May 1, 2026
Merged

Change commit message topic to 'container image digests'#127614
richlander merged 1 commit intomainfrom
richlander-patch-1

Conversation

@richlander
Copy link
Copy Markdown
Member

"dependencies" is confusing or misleading. That's the default renovate terminology. "digests" is much clearer to anyone that is in a position to merge the generated PRs.

PR examples: https://github.com/dotnet/runtime/pulls?q=is%3Apr+author%3Adotnet-renovate-bot+

@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Renovate’s commit message topic for pinned Docker image updates so the generated PR titles more accurately reflect that these changes are digest pins (rather than generic “dependencies”), improving clarity for reviewers/mergers.

Changes:

  • Renamed Renovate commitMessageTopic from “container image dependencies” to “container image digests” for the relevant package rule.

@github-actions
Copy link
Copy Markdown
Contributor

🤖 Copilot Code Review — PR #127614

Note

This review was generated by Copilot.

Holistic Assessment

Motivation: The PR improves the accuracy of Renovate bot commit messages. Since pinDigests: true means Renovate specifically pins container image SHA256 digests, "container image digests" is a more precise topic than the previous "container image dependencies."

Approach: Editing the commitMessageTopic string in eng/renovate.json is the correct and minimal way to achieve this.

Summary: ✅ LGTM. This is a single-word string change in a Renovate configuration file that makes auto-generated commit messages more descriptive. No correctness, performance, or compatibility concerns.


Detailed Findings

✅ Configuration Correctness

The change is valid Renovate configuration. The commitMessageTopic field accepts any string and is used as a template variable in Renovate's auto-generated commit messages. The new value "container image digests" accurately reflects what pinDigests: true does — it pins SHA256 digests for container images referenced in the matched pipeline YAML files.

✅ No Functional Impact

This change only affects the text of commit messages generated by the Renovate bot. It has no impact on which files are matched, how digests are pinned, or any runtime/build behavior. No new public API surface is introduced.

Generated by Code Review for issue #127614 ·

@richlander
Copy link
Copy Markdown
Member Author

/ba-g "Unrelated failures"

@richlander richlander enabled auto-merge (squash) May 1, 2026 03:54
@richlander richlander merged commit 0e1b3d9 into main May 1, 2026
162 of 165 checks passed
@richlander richlander deleted the richlander-patch-1 branch May 1, 2026 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants