Skip to content

Commit 8f4140f

Browse files
fix: update permissions and command execution for linting translation files (#22922)
* fix: make script for linting translations executable * fix: execute the command directly * feat: surface any errors if lint check fails * fix: do not return early if lint check errors Co-authored-by: Octomerger Bot <63058869+Octomerger@users.noreply.github.com>
1 parent f17c7a2 commit 8f4140f

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

script/i18n/lint-translation-files.js

100644100755
Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,10 +53,9 @@ function lintAndResetFiles(checkType) {
5353

5454
// Lint the files based on the check type and output the errors to a log file.
5555
try {
56-
execSync(cmd[checkType])
56+
execSync(cmd)
5757
} catch (error) {
58-
console.log(`There were new ${checkType} errors!`)
59-
return
58+
console.log(`There were new ${checkType} errors! Check ${log} for more details.`)
6059
}
6160

6261
// Reset the files

0 commit comments

Comments
 (0)