Skip to content

Comments

fix agent writer workflow#24199

Merged
dvdksn merged 2 commits intodocker:mainfrom
dvdksn:fix-agent-writer-workflow
Feb 23, 2026
Merged

fix agent writer workflow#24199
dvdksn merged 2 commits intodocker:mainfrom
dvdksn:fix-agent-writer-workflow

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Feb 23, 2026

  • ci: remove hardcoded cagent version from agent writer workflow
  • ci: use create-pull-request in writer agent workflow

Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Feb 23, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 6db352f
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/699c710b4070bb000843680f
😎 Deploy Preview https://deploy-preview-24199--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added area/release Relates to CI or deployment ci ci updates labels Feb 23, 2026
@dvdksn dvdksn requested a review from a team February 23, 2026 15:25
Copy link
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Review Summary

The workflow refactoring simplifies the PR creation logic by using the peter-evans/create-pull-request action, which is good. However, the new conditional logic has an issue with how it distinguishes between "no changes detected" (expected) and "action failure" (error). Both scenarios now trigger the same "no changes" comment, which could mask genuine action failures and reduce debugging visibility.

@dvdksn dvdksn merged commit b5e217c into docker:main Feb 23, 2026
18 checks passed
@dvdksn dvdksn deleted the fix-agent-writer-workflow branch February 23, 2026 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/release Relates to CI or deployment ci ci updates status/review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants