mirror of
https://github.com/github/codeql.git
synced 2025-12-16 08:43:11 +01:00
Remove trailing periods from @name metadata in query files
Fixed 73 .ql query files where the @name metadata contained an ending period. This ensures consistency with the CodeQL query metadata style guidelines.
This commit is contained in:
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Don't use getAQlClass.
|
||||
* @name Don't use getAQlClass
|
||||
* @description Any use of getAQlClass causes both compile-time and runtime to be significantly slower.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Var only used in one side of disjunct.
|
||||
* @name Var only used in one side of disjunct
|
||||
* @description Only using a variable on one side of a disjunction can cause a cartesian product.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Acronyms should be PascalCase/camelCase.
|
||||
* @name Acronyms should be PascalCase/camelCase
|
||||
* @description Acronyms should be PascalCase/camelCase instead of upper-casing all the letters.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Use of 'if' with a 'none()' branch.
|
||||
* @name Use of 'if' with a 'none()' branch
|
||||
* @description Using 'if p() then q() else none()' is bad style. It should be rewritten as 'p() and q()'.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Using a different parameter name than used in the super-predicate.
|
||||
* @name Using a different parameter name than used in the super-predicate
|
||||
* @description Using another parameter can be an indication of copy-pasted code, or a mistake.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Redundant assignment.
|
||||
* @name Redundant assignment
|
||||
* @description Assigning the same value twice is redundant.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Superfluous 'exists' conjunct.
|
||||
* @name Superfluous 'exists' conjunct
|
||||
* @description Writing 'exists(x)' when the existence of X is implied by another conjunct is bad practice.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Swapped parameter names in overriding predicate.
|
||||
* @name Swapped parameter names in overriding predicate
|
||||
* @description Swapping the parameter names in an overriding method indicates an implementation mistake.
|
||||
* @kind problem
|
||||
* @problem.severity error
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Suggest using non-extending subtype relationships.
|
||||
* @name Suggest using non-extending subtype relationships
|
||||
* @description Non-extending subtypes ("instanceof extensions") are generally preferable to instanceof expressions in characteristic predicates.
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Class QLDoc style.
|
||||
* @name Class QLDoc style
|
||||
* @description The QLDoc for a class should start with "A", "An", or "The".
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
/**
|
||||
* @name Predicate QLDoc style.
|
||||
* @name Predicate QLDoc style
|
||||
* @description The QLDoc for a predicate should start with "Gets" or "Holds".
|
||||
* @kind problem
|
||||
* @problem.severity warning
|
||||
|
||||
Reference in New Issue
Block a user