This is an automated email from the ASF dual-hosted git repository.
yao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 8c57500b6723 [SPARK-51632][INFRA] Passing Command Line Arguments of
lint-js to eslint
8c57500b6723 is described below
commit 8c57500b6723cc781efece7b6bff62e8eeee3730
Author: Kent Yao <y...@apache.org>
AuthorDate: Fri Mar 28 09:41:56 2025 +0800
[SPARK-51632][INFRA] Passing Command Line Arguments of lint-js to eslint
### What changes were proposed in this pull request?
This PR makes changes for Passing Command Line Arguments of lint-js to
eslint to make things possible like below,
- dev/lint-js --debug
- dev/lint-js --fix
- ...
### Why are the changes needed?
Improve dev experience for Spark UI
### Does this PR introduce _any_ user-facing change?
No, dev only
### How was this patch tested?
This PR does not expect all options passed function well but only useful
cases, like debug, fix etc.
### Was this patch authored or co-authored using generative AI tooling?
no
Closes #50430 from yaooqinn/SPARK-51632.
Authored-by: Kent Yao <y...@apache.org>
Signed-off-by: Kent Yao <y...@apache.org>
---
dev/lint-js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dev/lint-js b/dev/lint-js
index 1a94348b7430..6ec66df47a73 100755
--- a/dev/lint-js
+++ b/dev/lint-js
@@ -45,7 +45,7 @@ if ! npm ls eslint > /dev/null; then
npm ci eslint
fi
-npx eslint -c "$SPARK_ROOT_DIR/dev/eslint.js" ${LINT_TARGET_FILES[@]} | tee
"$LINT_JS_REPORT_FILE_NAME"
+npx eslint -c "$SPARK_ROOT_DIR/dev/eslint.js" "${LINT_TARGET_FILES[@]}" "$@" |
tee "$LINT_JS_REPORT_FILE_NAME"
lint_status=$?
if [ "$lint_status" = "0" ] ; then
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
