Skip to content

DOC: Update hard cap description for in-source comments - #6906

Merged
dzenanz merged 1 commit into
mainfrom
agents100_120
Sep 25, 2026
Merged

dzenanz merged 1 commit into
mainfrom
agents100_120

Conversation

@dzenanz

@dzenanz dzenanz commented Sep 25, 2026

Copy link
Copy Markdown
Member

Clarified the hard cap for in-source comments to prefer a short line: effectively increase from explicit 100, to 120 clang-format line length limit.

@github-actions github-actions Bot added type:Documentation Documentation improvement or change area:Documentation Issues affecting the Documentation module labels Sep 25, 2026
@greptile-apps

greptile-apps Bot commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low risk] Updates documentation about code comment guidelines.

No blocking issues remain.

Summary

The prose-budget table now states the in-source comment cap. No outstanding issues remain.

Reviews (2) · Last reviewed commit: "DOC: Update hard cap description for in-..."

Comment thread Documentation/AI/prose-budget.md Outdated
@greptile-apps

This comment has been minimized.

Clarified the hard cap for in-source comments:
increase from direct 100, to 120 clang-format line length limit.
@dzenanz

dzenanz commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

@greptileai re-review

@hjmjohnson
hjmjohnson self-requested a review September 25, 2026 14:24
@dzenanz
dzenanz merged commit 35b196e into main Sep 25, 2026
8 checks passed
@dzenanz
dzenanz deleted the agents100_120 branch September 25, 2026 14:32
@hjmjohnson

Copy link
Copy Markdown
Member

@dzenanz my refinement was minor. What you have is fine. No need ot revisit this now.

@dzenanz

dzenanz commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

Sorry, I did not see that you self-requested a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Documentation Issues affecting the Documentation module type:Documentation Documentation improvement or change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants