Skip to content

Commit d55d5cc

Browse files
committed
JS: Address comments from doc review
1 parent 66a16d2 commit d55d5cc

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

change-notes/1.24/analysis-javascript.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
| **Query** | **Tags** | **Purpose** |
1818
|---------------------------------------------------------------------------------|-------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
1919
| Cross-site scripting through exception (`js/xss-through-exception`) | security, external/cwe/cwe-079, external/cwe/cwe-116 | Highlights potential XSS vulnerabilities where an exception is written to the DOM. Results are not shown on LGTM by default. |
20-
| Regular expression always matches (`js/regex/always-matches`) | correctness, regular-expressions | Highlights regular expression checks that trivially succeed by matching an empty substring. |
20+
| Regular expression always matches (`js/regex/always-matches`) | correctness, regular-expressions | Highlights regular expression checks that trivially succeed by matching an empty substring. Results are shown on LGTM by default. |
2121

2222
## Changes to existing queries
2323

javascript/ql/src/RegExp/RegExpAlwaysMatches.qhelp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@
77
<p>
88
There are several built-in JavaScript functions that search for a regular expression match within a string,
99
such as <code>RegExp.prototype.test</code> and <code>String.prototype.search</code>.
10-
If the regular expression is not anchored, the regular expression does not need to match the whole string;
11-
it only needs to match a substring.
10+
If the regular expression is not anchored, it only needs to match a substring of the input
11+
and won't necessarily match the whole string.
1212
</p>
1313

1414
<p>

0 commit comments

Comments
 (0)