Skip to content

chore: clarify rule about trailing periods - #31175

Merged
mateo-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_humanize_claude
Jun 24, 2026
Merged

mateo-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_humanize_claude

Conversation

@mateo-berri

Copy link
Copy Markdown
Contributor

I notice Claude applies the rule irregularly. This aims to fix that

I notice Claude applies the rule irregularly. This aims to fix that
@mateo-berri
mateo-berri requested a review from Sameerlite June 24, 2026 04:55
@mateo-berri
mateo-berri enabled auto-merge (squash) June 24, 2026 04:56
@greptile-apps

greptile-apps Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR tightens the no-trailing-period writing guideline in CLAUDE.md so the intent is unambiguous: the rule applies to every paragraph, not only the final one in a document or comment.

  • The single changed line in CLAUDE.md appends two clarifying sentences to the existing bullet, spelling out scope ("every paragraph, not just the last one") and offering a rule of thumb ("unless there's a sentence immediately after, don't add a .").
  • The wording of the new sentences is internally consistent — mid-bullet sentences that are followed by another sentence do use a period, while the final sentence ends with a closing quotation mark, so the rule applies to itself correctly.

Confidence Score: 5/5

Only CLAUDE.md is touched; no production code, tests, or configuration files are affected.

A single-line clarification to a style guide with no code changes, no risk of behavioral regression, and the expanded wording correctly follows its own rule.

No files require special attention.

Important Files Changed

Filename Overview
CLAUDE.md Expands the trailing-period rule with an inline clarification stating the rule applies to every paragraph, plus a rule-of-thumb sentence; no logic or structural issues.

Reviews (1): Last reviewed commit: "chore: clarify rule about trailing perio..." | Re-trigger Greptile

@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mateo-berri
mateo-berri merged commit c0a1469 into litellm_internal_staging Jun 24, 2026
121 of 122 checks passed
@mateo-berri
mateo-berri deleted the litellm_humanize_claude branch June 24, 2026 05:06
@ishaan-berri ishaan-berri mentioned this pull request Jun 25, 2026
9 of 13 tasks
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