Skip to content

Update docs and samples to rename docker-agent, change usage samples to docker agent#1915

Merged
dgageot merged 1 commit intodocker:mainfrom
gtardif:rename_samples
Mar 4, 2026
Merged

Update docs and samples to rename docker-agent, change usage samples to docker agent#1915
dgageot merged 1 commit intodocker:mainfrom
gtardif:rename_samples

Conversation

@gtardif
Copy link
Contributor

@gtardif gtardif commented Mar 4, 2026

No description provided.

@gtardif gtardif requested a review from a team as a code owner March 4, 2026 09:51
Copy link

@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

Assessment: 🔴 CRITICAL

Found 2 issues in the documentation changes:

  • 1 HIGH severity issue: Command typo that will break user installation attempts
  • 1 MEDIUM severity issue: Incomplete documentation sentence

The rest of the PR (renaming cagent to docker-agent and updating command examples to docker agent) looks correct and consistent across all 100 files.

```bash
cd cagent
cagent run ./golang_developer.yaml
docker-agent run ./golang_developer.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

question: docker-agent run vs docker agent run?

Copy link
Contributor Author

@gtardif gtardif Mar 4, 2026

Choose a reason for hiding this comment

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

advertising docker agent run everywhere (except in the contibuting guidelines where we build and run ./bin/docker-agent for now)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

aheritier
aheritier previously approved these changes Mar 4, 2026
Copy link
Contributor

@aheritier aheritier left a comment

Choose a reason for hiding this comment

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

LGTM. Some files were reformatted but they are looking ok

…to `docker agent`

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>
@dgageot dgageot merged commit f6573da into docker:main Mar 4, 2026
5 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.

4 participants