Merge pull request #3510 from max-schaefer/cull-boring-queries

Approved by asgerf, esbena
2026-05-01 11:45:14 +02:00 · 2020-05-19 15:41:53 +01:00
parent f0f7e531d7 a803120414
commit 26dfca80f6
21 changed files with 43 additions and 20 deletions
--- a/javascript/ql/src/AngularJS/DeadAngularJSEventListener.ql
+++ b/javascript/ql/src/AngularJS/DeadAngularJSEventListener.ql
@@ -3,7 +3,7 @@
 * @description An AngularJS event listener that listens for a non-existent event has no effect.
 * @kind problem
 * @problem.severity warning
- * @precision medium
+ * @precision low
 * @id js/angular/dead-event-listener
 * @tags correctness
 *       frameworks/angularjs
--- a/javascript/ql/src/AngularJS/UnusedAngularDependency.ql
+++ b/javascript/ql/src/AngularJS/UnusedAngularDependency.ql
@@ -3,7 +3,7 @@
 * @description Unused dependencies are confusing, and should be removed.
 * @kind problem
 * @problem.severity recommendation
- * @precision high
+ * @precision low
 * @id js/angular/unused-dependency
 * @tags maintainability
 *       frameworks/angularjs
--- a/javascript/ql/src/DOM/ConflictingAttributes.ql
+++ b/javascript/ql/src/DOM/ConflictingAttributes.ql
@@ -8,7 +8,7 @@
 * @tags maintainability
 *       correctness
 *       external/cwe/cwe-758
- * @precision medium
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Declarations/DeadStoreOfGlobal.ql
+++ b/javascript/ql/src/Declarations/DeadStoreOfGlobal.ql
@@ -7,7 +7,7 @@
 * @tags maintainability
 *       correctness
 *       external/cwe/cwe-563
- * @precision medium
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Declarations/TooManyParameters.ql
+++ b/javascript/ql/src/Declarations/TooManyParameters.ql
@@ -6,7 +6,7 @@
 * @id js/too-many-parameters
 * @tags testability
 *       readability
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Declarations/UnusedProperty.ql
+++ b/javascript/ql/src/Declarations/UnusedProperty.ql
@@ -5,7 +5,7 @@
 * @problem.severity recommendation
 * @id js/unused-property
 * @tags maintainability
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Expressions/BitwiseSignCheck.ql
+++ b/javascript/ql/src/Expressions/BitwiseSignCheck.ql
@@ -6,7 +6,7 @@
 * @id js/bitwise-sign-check
 * @tags reliability
 *       correctness
- * @precision medium
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Expressions/CompareIdenticalValues.ql
+++ b/javascript/ql/src/Expressions/CompareIdenticalValues.ql
@@ -11,7 +11,7 @@
 *       convention
 *       external/cwe/cwe-570
 *       external/cwe/cwe-571
- * @precision medium
+ * @precision low
 */

 import Clones
--- a/javascript/ql/src/Expressions/MisspelledIdentifier.ql
+++ b/javascript/ql/src/Expressions/MisspelledIdentifier.ql
@@ -6,7 +6,7 @@
 * @id js/misspelled-identifier
 * @tags maintainability
 *       readability
- * @precision high
+ * @precision low
 */

 import Misspelling
--- a/javascript/ql/src/JSDoc/BadParamTag.ql
+++ b/javascript/ql/src/JSDoc/BadParamTag.ql
@@ -9,7 +9,7 @@
 * @tags maintainability
 *       readability
 *       documentation
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/JSDoc/JSDocForNonExistentParameter.ql
+++ b/javascript/ql/src/JSDoc/JSDocForNonExistentParameter.ql
@@ -8,7 +8,7 @@
 * @tags maintainability
 *       readability
 *       documentation
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/JSDoc/UndocumentedParameter.ql
+++ b/javascript/ql/src/JSDoc/UndocumentedParameter.ql
@@ -8,7 +8,7 @@
 * @tags maintainability
 *       readability
 *       documentation
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/LanguageFeatures/EmptyArrayInit.ql
+++ b/javascript/ql/src/LanguageFeatures/EmptyArrayInit.ql
@@ -7,7 +7,7 @@
 * @tags maintainability
 *       readability
 *       language-features
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/LanguageFeatures/SetterIgnoresParameter.ql
+++ b/javascript/ql/src/LanguageFeatures/SetterIgnoresParameter.ql
@@ -8,7 +8,7 @@
 * @tags reliability
 *       maintainability
 *       language-features
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/LanguageFeatures/WrongExtensionJSON.ql
+++ b/javascript/ql/src/LanguageFeatures/WrongExtensionJSON.ql
@@ -6,7 +6,7 @@
 * @id js/json-in-javascript-file
 * @tags maintainability
 *       language-features
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/NodeJS/CyclicImport.ql
+++ b/javascript/ql/src/NodeJS/CyclicImport.ql
@@ -8,7 +8,7 @@
 * @tags reliability
 *       maintainability
 *       frameworks/node.js
- * @precision medium
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/NodeJS/UnusedDependency.ql
+++ b/javascript/ql/src/NodeJS/UnusedDependency.ql
@@ -3,7 +3,7 @@
 * @description If unnecessary package dependencies are included in package.json, the
 *              package will become harder to install.
 * @kind problem
- * @problem.severity warning
+ * @problem.severity recommendation
 * @id js/node/unused-npm-dependency
 * @tags maintainability
 *       frameworks/node.js
--- a/javascript/ql/src/Statements/EphemeralLoop.ql
+++ b/javascript/ql/src/Statements/EphemeralLoop.ql
@@ -6,7 +6,7 @@
 * @problem.severity recommendation
 * @id js/single-run-loop
 * @tags readability
- * @precision high
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Statements/NestedLoopsSameVariable.ql
+++ b/javascript/ql/src/Statements/NestedLoopsSameVariable.ql
@@ -7,7 +7,7 @@
 * @id js/nested-loops-with-same-variable
 * @tags maintainability
 *       correctness
- * @precision medium
+ * @precision low
 */

 import javascript
--- a/javascript/ql/src/Statements/ReturnOutsideFunction.ql
+++ b/javascript/ql/src/Statements/ReturnOutsideFunction.ql
@@ -7,7 +7,7 @@
 * @id js/return-outside-function
 * @tags reliability
 *       correctness
- * @precision medium
+ * @precision low
 */

 import javascript