Skip to content

unquoted strings are fine until they contain special characters like :#1641

Merged
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:yaml-is-deceptively-simple
Feb 8, 2026
Merged

unquoted strings are fine until they contain special characters like :#1641
derekmisler merged 1 commit intodocker:mainfrom
derekmisler:yaml-is-deceptively-simple

Conversation

@derekmisler
Copy link
Contributor

No description provided.

@derekmisler derekmisler self-assigned this Feb 8, 2026
@derekmisler derekmisler marked this pull request as ready for review February 8, 2026 17:59
@derekmisler derekmisler requested a review from a team as a code owner February 8, 2026 17:59
Copy link
Contributor

@github-actions github-actions 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

LGTM - No issues found.

This PR correctly adds quotes around YAML string values containing colons. This is a valid YAML syntax improvement that prevents potential parsing ambiguities. The quoted strings will parse correctly and maintain the same semantic meaning.

Changed Lines Analysis:

  • Line 146: Added quotes to security agent description ✓
  • Line 210: Added quotes to bugs agent description ✓

Both changes are safe and improve configuration correctness.

@derekmisler derekmisler merged commit 3632c75 into docker:main Feb 8, 2026
11 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