docker-docs

mirror of https://github.com/docker/docs.git synced 2026-06-19 07:35:16 +00:00

Files

T

David Karlsson e80099f5bc Fix PR-review bot false positives (80-char rule, enable/disable) (#25237 )

## Summary

An audit of docker-agent review comments across recent merged PRs
surfaced two
recurring false positives, both traced to their source:

- The bot's prompt (`.github/workflows/pr-review.yml`) instructed it to
flag
lines over 80 characters, but no such rule exists — markdownlint MD013
is
  disabled and STYLE.md never mentions it. The bot itself conceded this
  mid-review on a past PR. Removed the instruction.
- STYLE.md's word list banned "enable" yet used it in a ✅ example, so
the bot
flagged every "enable"/"disable" as a violation. Added a note clarifying
that
"turn on/off" is for UI toggles while "enable/disable" is fine in
general
  prose.

## Learnings

- The docker-agent PR reviewer is driven by the `additional-prompt`
block in
  `.github/workflows/pr-review.yml` (via `docker/cagent-action`), plus
`STYLE.md` and `COMPONENTS.md` loaded as prompt files. That prompt — not
just
the style guides — is the source of truth for what the bot flags, so
review
  noise can be tuned there directly.

Generated by Claude Code

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

2026-06-03 08:10:35 +02:00

agents

Clarify docs-scanner broken link rule to prevent 404s being filed as cross-reference issues

2026-04-08 15:36:42 +02:00

ISSUE_TEMPLATE

Desktop: update issue tracker links (#23976 )

2026-01-19 11:08:38 +00:00

prompts

docs: fix freshness prompt spelling (#24841 )

2026-04-21 13:32:48 -07:00

workflows

Fix PR-review bot false positives (80-char rule, enable/disable) (#25237 )