Skip to content
Open
Show file tree
Hide file tree
Changes from 24 commits
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
4df54a2
Add failing tests for S2871 false positive
Feb 24, 2026
b5ca9a7
Fix FP for S2871: default sort for string arrays in specific contexts
Feb 24, 2026
cb84fe5
Update ruling results after S2871 FP fix
francois-mora-sonarsource Feb 25, 2026
20aed65
Fix SonarQube issues in S2871 implementation
Feb 26, 2026
3cf15a4
Address PR comment from francois-mora-sonarsource
Feb 26, 2026
2e12571
Restore its/ to match master by removing submodule entries
francois-mora-sonarsource Feb 26, 2026
681a7a5
Update ruling results
github-actions[bot] Feb 26, 2026
2a975dc
Fix build errors for S2871
Mar 10, 2026
65c0380
Merge remote-tracking branch 'origin/master' into fix/JS-1300-fix-fp-…
Mar 10, 2026
2ab0645
Update ruling results for PR #6463 (#6560)
github-actions[bot] Mar 10, 2026
828dac0
Address review comment from zglicz on packages/jsts/src/rules/S2871/r…
Mar 13, 2026
ec4658c
Fix S2871 rule: handle undefined node in isArrayFromKeyOrEntryCall
Mar 13, 2026
73dbd45
Fix SonarQube issues in S2871 rule implementation
Mar 13, 2026
6be5cb5
test: add tests to increase code coverage for S2871
Mar 13, 2026
9f4b4aa
Merge remote-tracking branch 'origin/master' into fix/JS-1300-fix-fp-…
Mar 16, 2026
cbad8c2
Merge remote-tracking branch 'origin/master' into fix/JS-1300-fix-fp-…
Mar 17, 2026
48f9e7e
Address review comment from francois-mora-sonarsource on packages/jst…
Mar 17, 2026
0f3573e
Address review comment from sonar-review-alpha on packages/jsts/src/r…
Mar 17, 2026
e75e581
Address review comment from sonar-review-alpha on packages/jsts/src/r…
Mar 17, 2026
3b33a79
Address review comment from sonar-review-alpha on packages/jsts/src/r…
Mar 17, 2026
85b6775
Merge remote-tracking branch 'origin/master' into fix/JS-1300-fix-fp-…
Mar 17, 2026
ba050f3
Sync expected ruling files after full refinement
Mar 17, 2026
563e0c2
Refine S2871: suppress sort() on for-in key collection pattern
Mar 17, 2026
830c0dc
Fix SonarQube issues in S2871 rule implementation
Mar 17, 2026
a96fcfe
Address review comment from sonar-review-alpha on packages/jsts/src/r…
Mar 17, 2026
5d55211
JS-1300 Update TS integration test for string array sort
francois-mora-sonarsource Mar 18, 2026
7f9d011
Address PR comment from francois-mora-sonarsource
Mar 18, 2026
658cc56
Use TypeScript type checker as primary mechanism in S2871
Mar 18, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions its/ruling/src/test/expected/Ghost/javascript-S2871.json

This file was deleted.

13 changes: 0 additions & 13 deletions its/ruling/src/test/expected/TypeScript/typescript-S2871.json
Original file line number Diff line number Diff line change
@@ -1,18 +1,5 @@
{
"TypeScript:src/compiler/tsc.ts": [
381,
381
],
"TypeScript:src/harness/fourslash.ts": [
2388,
2389
],
"TypeScript:src/server/editorServices.ts": [
594,
594,
1599
],
"TypeScript:src/server/utilities.ts": [
191
]
}
18 changes: 0 additions & 18 deletions its/ruling/src/test/expected/ace/javascript-S2871.json

This file was deleted.

5 changes: 0 additions & 5 deletions its/ruling/src/test/expected/angular.js/javascript-S2871.json

This file was deleted.

4 changes: 0 additions & 4 deletions its/ruling/src/test/expected/ant-design/javascript-S2871.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,6 @@
1469
],
"ant-design:components/tree/__tests__/util.test.js": [
24,
24
],
"ant-design:site/theme/template/Content/Demo/CodePreview.jsx": [
12
]
}
6 changes: 0 additions & 6 deletions its/ruling/src/test/expected/console/typescript-S2871.json

This file was deleted.

12 changes: 0 additions & 12 deletions its/ruling/src/test/expected/desktop/typescript-S2871.json
Original file line number Diff line number Diff line change
@@ -1,17 +1,5 @@
{
"desktop:app/src/lib/read-emoji.ts": [
120
],
"desktop:app/src/ui/acknowledgements/acknowledgements.tsx": [
65
],
"desktop:app/src/ui/history/commit-list.tsx": [
416
],
"desktop:script/draft-release/run.ts": [
95
],
"desktop:script/generate-octicons.ts": [
58
]
}
9 changes: 0 additions & 9 deletions its/ruling/src/test/expected/eigen/typescript-S2871.json

This file was deleted.

6 changes: 0 additions & 6 deletions its/ruling/src/test/expected/jquery/javascript-S2871.json

This file was deleted.

4 changes: 0 additions & 4 deletions its/ruling/src/test/expected/p5.js/javascript-S2871.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,6 @@
"p5.js:docs/yuidoc-p5-theme/assets/js/reference.js": [
4359
],
"p5.js:lib/addons/p5.sound.js": [
2835,
2898
],
"p5.js:src/utilities/array_functions.js": [
266
]
Expand Down
12 changes: 0 additions & 12 deletions its/ruling/src/test/expected/qunit/javascript-S2871.json

This file was deleted.

5 changes: 0 additions & 5 deletions its/ruling/src/test/expected/router/javascript-S2871.json

This file was deleted.

10 changes: 0 additions & 10 deletions its/ruling/src/test/expected/searchkit/typescript-S2871.json

This file was deleted.

5 changes: 0 additions & 5 deletions its/ruling/src/test/expected/underscore/javascript-S2871.json

This file was deleted.

7 changes: 3 additions & 4 deletions its/ruling/src/test/expected/vuetify/javascript-S2871.json
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
{
"vuetify:packages/docs/src/examples/v-treeview/misc-selectable-icons.vue": [
133
],
"vuetify:packages/docs/src/service-worker.js": [
84
],
"vuetify:packages/vuetify/build/rollup.config.js": [
174,
292
]
}
Loading
Loading