diff --git a/frontend/README.md b/frontend/README.md index 265149205..2eab25dbe 100644 --- a/frontend/README.md +++ b/frontend/README.md @@ -8,8 +8,13 @@ This allows npm packages within this monorepo to depend on each other. ## Linting We demonstrate the usage of [rules_lint]. There are a few ways to wire this up, we show two: -- in the `next.js/` folder, `npm run lint` does a `bazel build` with a config setting that makes the build fail when lint violations are found. -- in the `react` folder, an `eslint_test` target results in test failures when lint violations are found. +- *build failure*: in the `next.js/` folder, `npm run lint` does a `bazel build` with a config setting that makes the build fail when lint violations are found. +- *test failure*: in the `react` folder, an `eslint_test` target results in test failures when lint violations are found. + +However, in both cases this inhibits creation of new lint rules or even upgrading the linter, because it requires updating the entire repository to fix or suppress +new lint violations at the same time. +We recommend showing lint results during code review instead. +See [rules_js]: https://docs.aspect.build/rules/aspect_rules_js [rules_lint]: https://github.com/aspect-build/rules_lint