Skip to content

Commit e5d89b9

Browse files
authored
Merge pull request github#12780 from aibaars/shared-yaml-lib
JS: extract YAML library to a shared pack
2 parents f6d000e + 048fb8b commit e5d89b9

File tree

8 files changed

+730
-503
lines changed

8 files changed

+730
-503
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
---
2+
category: majorAnalysis
3+
---
4+
* The Yaml.qll library was moved into a shared library pack named `codeql/yaml` to make it possible for other languages to re-use it. This change should be backwards compatible for existing JavaScript queries.

javascript/ql/lib/qlpack.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,6 @@ dependencies:
99
codeql/regex: ${workspace}
1010
codeql/tutorial: ${workspace}
1111
codeql/util: ${workspace}
12+
codeql/yaml: ${workspace}
1213
dataExtensions:
1314
- semmle/javascript/frameworks/**/model.yml

javascript/ql/lib/semmle/javascript/PrintAst.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ private predicate isNotNeeded(Locatable el) {
5151
private string getQlClass(Locatable el) {
5252
result = "[" + el.getPrimaryQlClasses() + "] "
5353
// Alternative implementation -- do not delete. It is useful for QL class discovery.
54-
// not el.getAPrimaryQlClass() = "???" and result = "[" + getPrimaryQlClasses() + "] " or el.getAPrimaryQlClass() = "???" and result = "??[" + concat(el.getAQlClass(), ",") + "] "
54+
// not el.getAPrimaryQlClass() = "???" and result = "[" + el.getPrimaryQlClasses() + "] " or el.getAPrimaryQlClass() = "???" and result = "??[" + concat(el.getAQlClass(), ",") + "] "
5555
}
5656

5757
/**

0 commit comments

Comments
 (0)