Skip to content

Use marker-based PR comments in Docker workflow#158

Merged
nevoodoo merged 1 commit intomainfrom
fix/docker-pr-comment-marker
Apr 14, 2026
Merged

Use marker-based PR comments in Docker workflow#158
nevoodoo merged 1 commit intomainfrom
fix/docker-pr-comment-marker

Conversation

@nevoodoo
Copy link
Copy Markdown
Contributor

@nevoodoo nevoodoo commented Mar 30, 2026

Description

Currently the Docker workflow adds a comment and overwrites it on every push, but this is an issue when other workflows may add their own comments as it has no way of finding the existing comment and instead it just edits the last one on the PR.

Summary

  • Replaces gh pr comment --edit-last with HTML marker-based comment isolation to find the exact comment made by the job
  • Uses <!-- cpg-flow-docker --> marker to find and update only the Docker comment
  • Prevents the Docker workflow from overwriting comments posted by other workflows

Replace --edit-last with HTML marker (<!-- cpg-flow-docker -->) to
find and update the Docker comment specifically. Prevents clobbering
comments from other workflows.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nevoodoo nevoodoo requested a review from a team as a code owner March 30, 2026 02:43
@github-actions
Copy link
Copy Markdown
Contributor

🐳 Docker Image Built

A new Docker image has been built for this PR:

Image: australia-southeast1-docker.pkg.dev/cpg-common/images-dev/cpg_flow:0a7c20a02a38b3c85f3efd62f492b8e96280eb58

Pull command:

docker pull australia-southeast1-docker.pkg.dev/cpg-common/images-dev/cpg_flow:0a7c20a02a38b3c85f3efd62f492b8e96280eb58

🔗 View in Google Cloud Console


This comment was automatically generated by the Docker workflow.

@nevoodoo nevoodoo requested a review from rameshka March 30, 2026 02:49
Copy link
Copy Markdown
Contributor

@rameshka rameshka left a comment

Choose a reason for hiding this comment

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

Hey @nevoodoo, changes are looking good.

@nevoodoo nevoodoo merged commit df736db into main Apr 14, 2026
6 of 7 checks passed
@nevoodoo nevoodoo deleted the fix/docker-pr-comment-marker branch April 14, 2026 06:39
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.

2 participants