Merge branch 'arguments-keyword-sast' into 'master'

Don't use arguments keyword in gettext script

Closes #50457

See merge request gitlab-org/gitlab-ce!21296

2 files changed, 10 insertions, 5 deletions

diff --git a/changelogs/unreleased/arguments-keyword-sast.yml b/changelogs/unreleased/arguments-keyword-sast.yml
new file mode 100644
index 00000000000..2ecbc5e8174
--- /dev/null
+++ b/changelogs/unreleased/arguments-keyword-sast.yml
@@ -0,0 +1,5 @@
+---
+title: Don't use arguments keyword in gettext script
+merge_request: 21296
+author: gfyoung
+type: fixed
diff --git a/scripts/frontend/extract_gettext_all.js b/scripts/frontend/extract_gettext_all.js
index af8cc4c3341..725522a3540 100644
--- a/scripts/frontend/extract_gettext_all.js
+++ b/scripts/frontend/extract_gettext_all.js
@@ -7,7 +7,7 @@ const {
 } = require('gettext-extractor-vue');
 const ensureSingleLine = require('../../app/assets/javascripts/locale/ensure_single_line.js');
 
-const arguments = argumentsParser
+const args = argumentsParser
   .option('-f, --file <file>', 'Extract message from one single file')
   .option('-a, --all', 'Extract message from all js/vue files')
   .parse(process.argv);
@@ -61,12 +61,12 @@ function printJson() {
   console.log(JSON.stringify(messages));
 }
 
-if (arguments.file) {
-  vueParser.parseFile(arguments.file).then(() => printJson());
-} else if (arguments.all) {
+if (args.file) {
+  vueParser.parseFile(args.file).then(() => printJson());
+} else if (args.all) {
   vueParser.parseFilesGlob('{ee/app,app}/assets/javascripts/**/*.{js,vue}').then(() => printJson());
 } else {
   console.warn('ERROR: Please use the script correctly:');
-  arguments.outputHelp();
+  args.outputHelp();
   process.exit(1);
 }