jethac opened a new issue, #42981: URL: https://github.com/apache/superset/issues/42981
### Problem `superset-frontend/package.json` runs oxlint with `--quiet`: ``` oxlint --config oxlint.json --quiet oxlint --config oxlint.json --fix --quiet ``` `--quiet` suppresses warn-level diagnostics. Five rules are set to `warn` in `oxlint.json`, so none of them can fail CI or appear in the tech-debt metrics path: | rule | current count | | --- | --- | | `react-hooks/exhaustive-deps` | 381 | | `react/jsx-key` | 80 | | `react-hooks/rules-of-hooks` | 47 | | `no-console` | — | | `prefer-destructuring` | — | | `react/no-unstable-nested-components` | — | The first three are correctness rules, not style. A missing dependency in a `useEffect`/`useCallback`/`useMemo` array is a stale closure or an effect that fails to re-run, which users experience as "the chart doesn't update until I refresh". `rules-of-hooks` violations can throw outright — see apache/superset#42978. The consequence is that these counts move without anyone seeing. `exhaustive-deps` stopped being recorded in the tech-debt series on 2026-05-13 at 238 and measures 381 today, having grown roughly 60% while invisible to both CI and the dashboard. ### Proposed change Drop `--quiet` from the metrics path at minimum, so warn-level rules are counted and trended even if they do not gate. Gating them is a larger conversation given the current counts; being able to see them is not. Verified against `master` at `3539c41dab`. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
