Skip to content

Conversation

@nstepien
Copy link
Collaborator

@nstepien nstepien commented Feb 9, 2026

https://github.com/SonarSource/SonarJS/blob/master/packages/jsts/src/rules/README.md#rules
I don't know why, but it looks like the pages for each rule is behind auth, so I couldn't check them.
I've basically enabled almost every rules, disabled rules that were previously disabled, disabled the aws rules, and disabled new rules that seemed inconvenient.
I haven't checked other rules that have not been an issue in this repo, so there are probably some more rules that we may want to disable.

@nstepien nstepien self-assigned this Feb 9, 2026
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.61%. Comparing base (3c52d8f) to head (2eee2bb).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3953   +/-   ##
=======================================
  Coverage   97.61%   97.61%           
=======================================
  Files          36       36           
  Lines        1470     1470           
  Branches      477      477           
=======================================
  Hits         1435     1435           
  Misses         35       35           
Files with missing lines Coverage Δ
src/DataGrid.tsx 98.99% <100.00%> (ø)
src/GroupRow.tsx 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@nstepien nstepien marked this pull request as ready for review February 9, 2026 15:06
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