Skip to content

docs: tighten comment guidance in AGENTS.md#3380

Merged
Sayt-0 merged 1 commit into
docker:mainfrom
dgageot:worktree-board-d6925c90f2f80556
Jul 1, 2026
Merged

docs: tighten comment guidance in AGENTS.md#3380
Sayt-0 merged 1 commit into
docker:mainfrom
dgageot:worktree-board-d6925c90f2f80556

Conversation

@dgageot

@dgageot dgageot commented Jul 1, 2026

Copy link
Copy Markdown
Member

The existing Code Comments Philosophy section explains when to comment, but said nothing about comment length. In practice, verbose block comments and prose-style notes have been creeping in despite the guidance.

This adds a short list under the existing paragraph that reinforces brevity: prefer a single line, cut filler, write terse notes rather than prose, and treat the need for a paragraph-long comment as a signal to refactor instead.

@dgageot dgageot requested a review from a team as a code owner July 1, 2026 14:14

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

@Sayt-0 Sayt-0 merged commit 0e4c9b4 into docker:main Jul 1, 2026
9 of 10 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.

3 participants