Skip to content

chore: add ESLint formatting rules and lint:fix script#51

Merged
regevbr merged 2 commits intomasterfrom
chore/add-eslint-formatting-rules
Feb 1, 2026
Merged

chore: add ESLint formatting rules and lint:fix script#51
regevbr merged 2 commits intomasterfrom
chore/add-eslint-formatting-rules

Conversation

@regevbr
Copy link
Copy Markdown
Owner

@regevbr regevbr commented Feb 1, 2026

Summary

  • Add comprehensive ESLint formatting rules for consistent code style
  • Add lint:fix script to package.json for auto-fixing issues
  • Add pnpm-workspace.yaml for pnpm workspace configuration
  • Auto-fix all existing formatting issues across the codebase

New ESLint Rules

Whitespace: no-trailing-spaces, eol-last, no-multiple-empty-lines, no-multi-spaces

Spacing: keyword-spacing, space-before-blocks, space-infix-ops, comma-spacing, semi-spacing, key-spacing, object-curly-spacing, array-bracket-spacing, space-in-parens, computed-property-spacing

Style: brace-style, curly, arrow-spacing, template-curly-spacing, rest-spread-spacing, prefer-const, no-var, eqeqeq

Test plan

  • Run pnpm run lint - should pass with no errors
  • Run pnpm run test - all tests should pass
  • Run pnpm run ci - full CI pipeline should pass

🤖 Generated with Claude Code

Add comprehensive ESLint rules for consistent code formatting:
- Whitespace: no-trailing-spaces, eol-last, no-multiple-empty-lines
- Spacing: keyword-spacing, object-curly-spacing, comma-spacing, etc.
- Style: brace-style, curly, prefer-const, no-var, eqeqeq

Also adds lint:fix script and pnpm-workspace.yaml.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@regevbr regevbr enabled auto-merge (squash) February 1, 2026 06:48
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@qltysh
Copy link
Copy Markdown

qltysh Bot commented Feb 1, 2026

Qlty

Coverage Impact

This PR will not change total coverage.

Modified Files with Diff Coverage (6)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
src/lib/helpers.ts100.0%
Coverage rating: A Coverage rating: A
src/lib/rulesEngine.ts100.0%
Coverage rating: A Coverage rating: A
src/lib/evaluator.ts100.0%
Coverage rating: A Coverage rating: A
src/lib/engine.ts100.0%
Coverage rating: A Coverage rating: A
src/lib/expressionHandler.ts100.0%
Coverage rating: A Coverage rating: A
src/lib/engineConsequenceEvaluator.ts100.0%
Total100.0%
🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@regevbr regevbr merged commit f3cf5c7 into master Feb 1, 2026
14 checks passed
@regevbr regevbr deleted the chore/add-eslint-formatting-rules branch February 1, 2026 06:52
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.

1 participant