pierrejeambrun commented on code in PR #42184:
URL: https://github.com/apache/airflow/pull/42184#discussion_r1758499310
##########
airflow/ui/package.json:
##########
@@ -7,7 +7,7 @@
"scripts": {
"dev": "vite",
"build": "tsc -b && vite build",
- "lint": "eslint . && tsc --p tsconfig.app.json",
+ "lint": "eslint --quiet && tsc --p tsconfig.app.json",
Review Comment:
Got it thanks.
Maybe we can just do the same that we have for the `www` which is "no
warning policy". We try to fix all and fail linting if there are warnings with
`--max-warnings=0`. Having tons of warnings is not great anyway so forcing
people to fix them along the way is practical.
Or maybe at least just remove the `--quiet` because I personally use both
CLI + IDE to fix and check for linting errors / warnings and if they report
different things it will drive me crazy 😄 (CLI omits warning but my IDE shows
something else)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
