Merge pull request #36296 from BerriAI/litellm_claude_md_descending_importance

docs: clarify guideline priority ordering in CLAUDE.md
This commit is contained in:
Mateo Wang 2026-08-08 13:01:38 -07:00 • committed by GitHub
commit 554f065361
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -9,7 +9,7 @@ Don't assume that the existing code is correct or the right way of doing things
- easy to maintain/change
- modern
In that order of importance
In descending order of importance
When adding new features, add meaningful tests. Don't add tests that don't check anything substantial and is there just to make the code coverage pass. Yes, code coverage is important, but I'd rather have no signal whether the code is working than tests that don't fail when code is broken. The goal is to have tests that would fail before the feature was added/if the code was mutated in a way that breaks the feature and succeed only when the feature is fully working. I should run mutation testing and see > 90% kill rate