Tag quality queries that are high precision and dont use pointsto

Author: joefarebrother

python/ql/src/Exceptions/CatchingBaseException.ql

@@ -2,9 +2,9 @@
  * @name Except block handles 'BaseException'
  * @description Handling 'BaseException' means that system exits and keyboard interrupts may be mis-handled.
  * @kind problem
- * @tags reliability
- *       readability
- *       convention
+ * @tags quality
+ *       reliability
+ *       error-handling
  *       external/cwe/cwe-396
  * @problem.severity recommendation
  * @sub-severity high

python/ql/src/Exceptions/EmptyExcept.ql

@@ -2,8 +2,9 @@
  * @name Empty except
  * @description Except doesn't do anything and has no comment
  * @kind problem
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       error-handling
  *       external/cwe/cwe-390
  * @problem.severity recommendation
  * @sub-severity high

python/ql/src/Expressions/CallToSuperWrongClass.ql

@@ -2,9 +2,9 @@
  * @name First argument to super() is not enclosing class
  * @description Calling super with something other than the enclosing class may cause incorrect object initialization.
  * @kind problem
- * @tags reliability
- *       maintainability
- *       convention
+ * @tags quality
+ *       reliability
+ *       correctness
  *       external/cwe/cwe-687
  * @problem.severity error
  * @sub-severity low

python/ql/src/Expressions/ExplicitCallToDel.ql

@@ -2,7 +2,8 @@
  * @name `__del__` is called explicitly
  * @description The `__del__` special method is called by the virtual machine when an object is being finalized. It should not be called explicitly.
  * @kind problem
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       correctness
  * @problem.severity warning
  * @sub-severity low

python/ql/src/Expressions/IncorrectComparisonUsingIs.ql

@@ -2,7 +2,8 @@
  * @name Comparison using is when operands support `__eq__`
  * @description Comparison using 'is' when equivalence is not the same as identity
  * @kind problem
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       correctness
  * @problem.severity warning
  * @sub-severity low

python/ql/src/Functions/ConsistentReturns.ql

@@ -2,8 +2,9 @@
  * @name Explicit returns mixed with implicit (fall through) returns
  * @description Mixing implicit and explicit returns indicates a likely error as implicit returns always return 'None'.
  * @kind problem
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
  * @problem.severity recommendation
  * @sub-severity high
  * @precision high

python/ql/src/Functions/InitIsGenerator.ql

@@ -2,7 +2,8 @@
  * @name `__init__` method is a generator
  * @description `__init__` method is a generator.
  * @kind problem
- * @tags reliability
+ * @tags quality
+ *       reliability
  *       correctness
  * @problem.severity error
  * @sub-severity low

python/ql/src/Functions/ModificationOfParameterWithDefault.ql

@@ -3,8 +3,9 @@
  * @description Modifying the default value of a parameter can lead to unexpected
  *              results.
  * @kind path-problem
- * @tags reliability
- *       maintainability
+ * @tags quality
+ *       reliability
+ *       correctness
  * @problem.severity error
  * @sub-severity low
  * @precision high