Exclusion filter should exclude only JS/TS files and patterns should …

…work only on relative path (#2615) * Match exclude patterns only on relative path of input file * Remove **/build/** pattern * Exclude only JS/TS files * update ruling * update docs
SonarSource · May 3, 2021 · 288a72f · 288a72f
1 parent ce70dda
commit 288a72f
Show file tree

Hide file tree

Showing 83 changed files with 1,130 additions and 6 deletions.
diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S105.json b/its/ruling/src/test/expected/js/jquery/javascript-S105.json
@@ -2,6 +2,45 @@
 'jquery:Gruntfile.js':[
 4,
 ],
+'jquery:build/release.js':[
+7,
+],
+'jquery:build/release/cdn.js':[
+4,
+],
+'jquery:build/release/dist.js':[
+5,
+],
+'jquery:build/tasks/amd.js':[
+8,
+],
+'jquery:build/tasks/build.js':[
+10,
+],
+'jquery:build/tasks/dist.js':[
+4,
+],
+'jquery:build/tasks/lib/rollup-plugin-file-overrides.js':[
+11,
+],
+'jquery:build/tasks/lib/spawn_test.js':[
+7,
+],
+'jquery:build/tasks/node_smoke_tests.js':[
+4,
+],
+'jquery:build/tasks/promises_aplus_tests.js':[
+4,
+],
+'jquery:build/tasks/qunit_fixture.js':[
+6,
+],
+'jquery:build/tasks/sourcemap.js':[
+6,
+],
+'jquery:build/tasks/testswarm.js':[
+4,
+],
 'jquery:src/ajax.js':[
 15,
 ],

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S106.json b/its/ruling/src/test/expected/js/jquery/javascript-S106.json
@@ -1,4 +1,18 @@
 {
+'jquery:build/release/cdn.js':[
+64,
+69,
+],
+'jquery:build/release/dist.js':[
+30,
+37,
+106,
+107,
+114,
+120,
+122,
+135,
+],
 'jquery:test/middleware-mockserver.js':[
 248,
 ],

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S109.json b/its/ruling/src/test/expected/js/jquery/javascript-S109.json
@@ -3,6 +3,23 @@
 174,
 174,
 ],
+'jquery:build/release/dist.js':[
+52,
+],
+'jquery:build/tasks/build.js':[
+122,
+],
+'jquery:build/tasks/dist.js':[
+46,
+48,
+48,
+],
+'jquery:build/tasks/testswarm.js':[
+20,
+20,
+20,
+28,
+],
 'jquery:src/ajax.js':[
 465,
 738,

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S1117.json b/its/ruling/src/test/expected/js/jquery/javascript-S1117.json
@@ -1,4 +1,16 @@
 {
+'jquery:build/release/cdn.js':[
+62,
+104,
+108,
+],
+'jquery:build/tasks/build.js':[
+118,
+315,
+],
+'jquery:build/tasks/dist.js':[
+31,
+],
 'jquery:src/attributes/attr.js':[
 116,
 ],

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S1121.json b/its/ruling/src/test/expected/js/jquery/javascript-S1121.json
@@ -1,4 +1,7 @@
 {
+'jquery:build/tasks/build.js':[
+95,
+],
 'jquery:src/ajax.js':[
 73,
 77,

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S126.json b/its/ruling/src/test/expected/js/jquery/javascript-S126.json
@@ -1,4 +1,7 @@
 {
+'jquery:build/release/cdn.js':[
+52,
+],
 'jquery:src/ajax.js':[
 101,
 231,

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S134.json b/its/ruling/src/test/expected/js/jquery/javascript-S134.json
@@ -1,4 +1,9 @@
 {
+'jquery:build/tasks/build.js':[
+95,
+130,
+150,
+],
 'jquery:src/ajax.js':[
 237,
 265,

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S138.json b/its/ruling/src/test/expected/js/jquery/javascript-S138.json
@@ -2,6 +2,9 @@
 'jquery:Gruntfile.js':[
 3,
 ],
+'jquery:build/tasks/build.js':[
+9,
+],
 'jquery:src/ajax.js':[
 382,
 ],
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,6 +2,9 @@ @@
     'jquery:Gruntfile.js':[
 ,
     ],
+    'jquery:build/tasks/build.js':[
+,
+    ],
     'jquery:src/ajax.js':[
 ,
     ],
@@ Expand Down @@