Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: tighten doc-check comment rules and shorten its prompt - #29231

Merged
bpmct merged 2 commits into
mainfrom
ben/doc-check-comment-rules
Sep 11, 2026
Merged

fix: tighten doc-check comment rules and shorten its prompt#29231
bpmct merged 2 commits into
mainfrom
ben/doc-check-comment-rules

Conversation

@bpmct

@bpmct bpmct commented Sep 11, 2026

Copy link
Copy Markdown
Member

Reviews posted a relative docs link that 404s on GitHub, an anchor that only exists after the PR merges, optional "consider" items as checkboxes, and two sticky comments on one PR. The rules now live in the skill, so the workflow prompt carries only what changes per run.

Generated by Coder Agents on behalf of @bpmct.

@DevelopmentCats DevelopmentCats 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.

LGTM

@bpmct
bpmct enabled auto-merge (squash) September 11, 2026 13:07
@bpmct
bpmct merged commit 5c73fa6 into main Sep 11, 2026
28 checks passed
@bpmct
bpmct deleted the ben/doc-check-comment-rules branch September 11, 2026 13:16
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 11, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants