chore: remove --max-warnings=0 from eslint commands - #8
Conversation
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
package.json— dropped--max-warnings=0fromlint,lint:fix, andsync:sponsors, and from the lint-staged*entry. Warnings no longer fail these commands; only errors do..claude/settings.json— dropped--max-warnings=0from the PostToolUse eslint--fixhook, matching the package scripts.
I verified the sweep is complete on both sides of the diff: the base had exactly five --max-warnings=0 invocations (the four in package.json plus the Claude hook), all five are covered here, and the head tree has no remaining occurrences anywhere (including .zed/settings.json, dependabot.yml, and docs). CI (ci.yaml runs pnpm run lint) and the sponsors sync workflow are consistent with the new behavior, and eslint.config.js sets no warning thresholds that would conflict. The change is mechanical, internally consistent, and matches the stated intent — one deliberate trade-off to note is that warnings can now accumulate silently without an alternative gate, which is the point of the change.
DeepSeek Flash (default — pick a model for stronger reviews) | 𝕏

Drops
--max-warnings=0from every eslint invocation so lint warnings no longer fail the check; only errors do.Applies to the
lint,lint:fix, andsync:sponsorsscripts, the lint-staged pre-commit command, and the Claude Code post-edit hook. Same change is being applied to middleapi/orpc and middleapi/standardserver.