Lint Regression
npm run lint (which runs npx eslint 'src/**/*.ts' --max-warnings 110) fails with exit code 1 because there are now 118 warnings, exceeding the configured threshold of 110 by 8.
Error Output
✖ 118 problems (0 errors, 118 warnings)
ESLint found too many warnings (maximum: 110).
Breakdown
| Category |
Count |
Location |
no-explicit-any |
~80 |
Test files (__tests__/) |
no-unused-vars |
~30 |
Test files (__tests__/) |
no-unused-vars (production) |
1 |
src/components/toast/toast.ts:538 |
Production Code Issue
src/components/toast/toast.ts:538 — unused position parameter in forEach callback:
this.containerMap.forEach((container, position) => {
container.remove();
});
Should be renamed to _position to match the unused-args pattern.
Suggested Fix
- Production code: Rename
position to _position in toast.ts:538
- Test cleanup: Remove unused imports and variables in test files to bring the count back under 110, or bump
--max-warnings if the current set is considered acceptable
Environment
- Branch: main (74a8e60)
- Node: current
- All tests pass (712/712), TypeScript clean, build succeeds
Lint Regression
npm run lint(which runsnpx eslint 'src/**/*.ts' --max-warnings 110) fails with exit code 1 because there are now 118 warnings, exceeding the configured threshold of 110 by 8.Error Output
Breakdown
no-explicit-any__tests__/)no-unused-vars__tests__/)no-unused-vars(production)src/components/toast/toast.ts:538Production Code Issue
src/components/toast/toast.ts:538— unusedpositionparameter inforEachcallback:Should be renamed to
_positionto match the unused-args pattern.Suggested Fix
positionto_positionintoast.ts:538--max-warningsif the current set is considered acceptableEnvironment