Skip to content

Commit

Permalink
Ignore CodeQL rules in external file (#384)
Browse files Browse the repository at this point in the history
* [Feature] Using CodeQL alert suppressions rather than LGTM alert suppressions
Fixes #383

* [Feature] Using CodeQL alert suppressions rather than LGTM alert suppressions
Fixes #383

* update comment
  • Loading branch information
benjaminbi authored Dec 1, 2022
1 parent e66ac40 commit bb103db
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 0 additions & 6 deletions .CodeQL.yml

This file was deleted.

2 changes: 1 addition & 1 deletion src/plugins/cordova-plugin-globalization/moment.js
Original file line number Diff line number Diff line change
Expand Up @@ -1156,7 +1156,7 @@
case 'Do':
return strict ? config._locale._ordinalParse : config._locale._ordinalParseLenient;
default :
a = new RegExp(regexpEscape(unescapeFormat(token.replace('\\', '')), 'i'));
a = new RegExp(regexpEscape(unescapeFormat(token.replace('\\', '')), 'i')); // CodeQL [js/incomplete-sanitization] This is from Cordova Plugin official code. The logic is right.
return a;
}
}
Expand Down

0 comments on commit bb103db

Please sign in to comment.