opensourcemirror
/
cockpit
mirror of https://github.com/cockpit-project/cockpit.git
1
0
Fork 0
Code Issues Releases Wiki Activity

test: apply eslint to .js files as well 

We want to lint all our JavaScript code, this would trip over
tools/webpack-make.js as it's a shell script. Ignorning it still gives
an eslint warning so this has to be converted to a proper JavaScript
file.
This commit is contained in:
Jelle van der Waa 2023-10-27 12:22:11 +02:00 committed by Jelle van der Waa
parent 0f31099539
commit cbfb1a0f43
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
test/static-code
View File

@ -20,11 +20,11 @@ set -eu
# note: `set -e` is not active during the tests.
find_scripts() {
# Helper to find all Python files in the tree
# Helper to find all scripts in the tree
(
# Any non-binary file which contains a python3 shebang
# Any non-binary file which contains a given shebang
git grep --cached -lIz '^#!.*'"$1"
# Any file ending in '.py'
# Any file ending in the provided regex
git ls-files -z "$2"
) | sort -z | uniq -z
}
@ -69,7 +69,7 @@ test_js_translatable_strings() {
if [ "${WITH_PARTIAL_TREE:-0}" = 0 ]; then
test_eslint() {
test -x node_modules/.bin/eslint -a -x /usr/bin/node || skip 'no eslint'
find_scripts 'node' '*.js?' | xargs -0 node_modules/.bin/eslint
find_scripts 'node' '*.js' '*.js?' | xargs -0 node_modules/.bin/eslint
}
fi

6
tools/webpack-make.js
View File

@ -1,3 +1,3 @@
#!/bin/sh
echo "This got renamed to ./build.js. Please see HACKING.md"
exit 1
#!/bin/env node
console.error("This got renamed to ./build.js. Please see HACKING.md");
process.exit(1);