Skip to content

Conversation

@pawelangelow
Copy link
Contributor

@pawelangelow pawelangelow commented Jan 9, 2026

What

const somethingNotUsed = 1; <--- is going to be reported as not used

... some code that does not use that variable ...

end

The meaningful commits are:

Testing

yarn lint


Note

Enforces unused-variable linting and cleans up codebase to comply.

  • Introduces shared noUnusedVarsConfig in .eslintrc.js and applies @typescript-eslint/no-unused-vars across API/UI overrides (allowing _-prefixed ignores and destructured arrays)
  • Refactors tests and UI code to satisfy the rule: remove unused imports/vars, rename unused params to _name/_ref/_className, simplify callbacks/handlers, and tweak a few helper signatures (e.g., getCommandsHistory, cloud constants) without changing behavior

Written by Cursor Bugbot for commit 8ef419d. This will update automatically on new commits. Configure here.

@pawelangelow pawelangelow self-assigned this Jan 9, 2026
@pawelangelow pawelangelow added the run-all-tests Trigger the CI to run the front-end, back-end and integration tests label Jan 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

Code Coverage - Integration Tests

Status Category Percentage Covered / Total
🟢 Statements 81.51% 16351/20059
🟡 Branches 64.6% 7424/11491
🟡 Functions 70.4% 2289/3251
🟢 Lines 81.15% 15385/18958

@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

Code Coverage - Backend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 92.42% 13988/15136
🟡 Branches 74.42% 4280/5751
🟢 Functions 86.01% 2151/2501
🟢 Lines 92.23% 13375/14502

Test suite run success

3014 tests passing in 288 suites.

Report generated by 🧪jest coverage report action from 45c8287

@pawelangelow pawelangelow added run-frontend-tests Trigger the CI to run the front-end tests and removed run-all-tests Trigger the CI to run the front-end, back-end and integration tests labels Jan 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 9, 2026

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.85% 21097/25463
🟡 Branches 68.07% 8883/13050
🟡 Functions 78.04% 5781/7408
🟢 Lines 83.26% 20665/24820

Test suite run success

5487 tests passing in 704 suites.

Report generated by 🧪jest coverage report action from 8ef419d

@pawelangelow pawelangelow merged commit 693eabb into main Jan 9, 2026
19 of 20 checks passed
@pawelangelow pawelangelow deleted the dev/enable-unused-vars-rule branch January 9, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-frontend-tests Trigger the CI to run the front-end tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants