Skip to content

chore: remove pre-commit rule - #36295

Merged
mateo-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_remove_pre_commit_rule
Aug 8, 2026
Merged

chore: remove pre-commit rule#36295
mateo-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_remove_pre_commit_rule

Conversation

@mateo-berri

Copy link
Copy Markdown
Contributor

Some users do not use make pre-commit as it is a multi-minute process. I personally use it, but I want users themselves to decide whether to pre-commit before each commit or not, based on what works best for them

some users do not use make pre-commit as it is a multi-minute process. I personally use it but I want users themselves to decide whether to pre-commit before each commit or not, based on what works best for them
@greptile-apps

greptile-apps Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR makes the CI-equivalent local check optional rather than requiring it before every commit.

  • Removes the instruction to run make check before every commit, including merge commits.
  • Retains guidance about the command alias and its saved log output.

Confidence Score: 5/5

The PR appears safe to merge because it makes an intentionally documented workflow-policy change without affecting runtime code or introducing contradictory guidance.

The sole change removes a deliberately unwanted mandate while retaining accurate information about invoking and inspecting the optional check.

Important Files Changed

Filename Overview
CLAUDE.md Removes the mandatory per-commit validation policy while preserving accurate operational guidance for developers who choose to run the check.

Reviews (1): Last reviewed commit: "chore: remove pre-commit rule" | Re-trigger Greptile

@mateo-berri
mateo-berri enabled auto-merge August 8, 2026 19:40
@mateo-berri
mateo-berri merged commit 334e6da into litellm_internal_staging Aug 8, 2026
76 checks passed
@mateo-berri
mateo-berri deleted the litellm_remove_pre_commit_rule branch August 8, 2026 19:44
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