Skip to content

update cagent-action to latest (with better permissions)#24523

Merged
dvdksn merged 1 commit intodocker:mainfrom
derekmisler:update-cagent-action-to-latest-with-better-permis
Mar 25, 2026
Merged

update cagent-action to latest (with better permissions)#24523
dvdksn merged 1 commit intodocker:mainfrom
derekmisler:update-cagent-action-to-latest-with-better-permis

Conversation

@derekmisler
Copy link
Copy Markdown
Contributor

@derekmisler derekmisler commented Mar 25, 2026

Description

Updates .github/workflows/pr-review.yml to pin docker/cagent-action's reusable review workflow to v1.3.1 by SHA, switch from secrets: inherit to explicit secret passing.

Related issues or tickets

Reviews

  • Technical review

Closes: https://github.com/docker/gordon/issues/311

@derekmisler derekmisler marked this pull request as ready for review March 25, 2026 13:58
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 25, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 23e0b78
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69c3eecd3975f6000843419e
😎 Deploy Preview https://deploy-preview-24523--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.

Signed-off-by: Derek Misler <derek.misler@docker.com>
@derekmisler derekmisler force-pushed the update-cagent-action-to-latest-with-better-permis branch from 5b6af7a to 23e0b78 Compare March 25, 2026 14:18
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

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

LGTM!

@dvdksn dvdksn merged commit e2ba7b6 into docker:main Mar 25, 2026
14 checks passed
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