Skip to content

chore: fix ESLint config (stack 2/3) - #8

Merged
ksambhi merged 1 commit into
fix/testsfrom
fix/lint
Oct 22, 2026
Merged

ksambhi merged 1 commit into
fix/testsfrom
fix/lint

Conversation

@ksambhi

@ksambhi ksambhi commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Stacked on the tests PR (base: fix/tests). Part 2/3.

ESLint crashed because .eslintrc.json referenced @nx/eslint-plugin, which isn't installed (leftover from an Nx setup). The per-package .eslintrc.json files also extended a nonexistent ../../.eslintrc.json.

  • Replace the root config with eslint:recommended + @typescript-eslint/recommended + prettier
  • Remove the per-package eslint configs and ignore files
  • The root .eslintignore now ignores node_modules, dist, coverage and *.d.ts

Verified locally: npm run lint lints 75 files with 0 errors and 0 warnings.


Generated by Claude Code

@ksambhi
ksambhi force-pushed the fix/lint branch 2 times, most recently from 94ece6e to 5e80b88 Compare October 22, 2026 11:07
@ksambhi
ksambhi force-pushed the fix/tests branch 2 times, most recently from 3fb8624 to 0aaa4ad Compare October 22, 2026 11:08
@ksambhi
ksambhi force-pushed the fix/lint branch 2 times, most recently from 8a16c75 to fddc64b Compare October 22, 2026 12:16
@github-actions

Copy link
Copy Markdown

Coverage report

St.❔
Category Percentage Covered / Total
🟢 Statements 82.44% 582/706
🔴 Branches 55.26% 105/190
🟡 Functions 77.23% 78/101
🟢 Lines 82.65% 562/680

Test suite run success

86 tests passing in 17 suites.

Report generated by 🧪jest coverage report action from 3d3763b

@ksambhi
ksambhi merged commit 9a606d7 into fix/tests Oct 22, 2026
3 of 7 checks passed
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