[CI] [Lint] [Dashboard] Make frontend linter automatically fix lint i…

…ssues (#43515) When making dashboard frontend changes, the linter outputs things like ` 0 errors and 7 warnings potentially fixable with the `--fix` option.` This PR adds the `--fix` command to the linter script to automatically fix lint issues when possible. This is consistent with how the other linters work in the script for non-frontend code. Signed-off-by: Archit Kulkarni <[email protected]>
ray-project · Mar 11, 2024 · a949436 · a949436
1 parent e9dd050
commit a949436
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/ci/lint/format.sh b/ci/lint/format.sh
@@ -208,7 +208,7 @@ format_frontend() {
  local filenames
  # shellcheck disable=SC2207
  filenames=($(find "${folder}"/src -name "*.ts" -or -name "*.tsx"))
- "${folder}/"node_modules/.bin/eslint --max-warnings 0 "${filenames[@]}"
+ "${folder}/"node_modules/.bin/eslint --fix --max-warnings 0 "${filenames[@]}"
  "${folder}/"node_modules/.bin/prettier -w "${filenames[@]}"
  "${folder}/"node_modules/.bin/prettier --check "${folder}/"public/index.html
  )