feat: stylelint (#341)

* feat: demonstrate stylelint

follows https://github.com/aspect-build/rules_lint/blob/main/lint/README.md#step-1-run-linter-in-the-example

* feat: create stylelint linter aspect

Follows instructions: https://github.com/aspect-build/rules_lint/blob/main/lint/README.md#step-2-create-linter

* chore: add stylelint to README

* chore: cleanups for review

* chore: more code review cleanup

README.md

@@ -25,7 +25,7 @@ New tools are being added frequently, so check this page again!
 | ---------------------- | --------------------- | ---------------- |
 | C / C++                | [clang-format]        | [clang-tidy]     |
 | Cuda                   | [clang-format]        |                  |
-| CSS, Less, Sass        | [Prettier]            |                  |
+| CSS, Less, Sass        | [Prettier]            | [Stylelint]      |
 | Go                     | [gofmt] or [gofumpt]  |                  |
 | GraphQL                | [Prettier]            |                  |
 | HCL (Hashicorp Config) | [terraform] fmt       |                  |
@@ -73,6 +73,7 @@ New tools are being added frequently, so check this page again!
 [vale]: https://vale.sh/
 [yamlfmt]: https://github.com/google/yamlfmt
 [rustfmt]: https://rust-lang.github.io/rustfmt
+[stylelint]: https://stylelint.io
 
 1. Non-hermetic: requires that a swift toolchain is installed on the machine.
    See https://github.com/bazelbuild/rules_swift#1-install-swift

docs/BUILD.bazel

@@ -32,6 +32,11 @@ stardoc_with_diff_test(
     bzl_library_target = "//format:defs",
 )
 
+stardoc_with_diff_test(
+    name = "stylelint",
+    bzl_library_target = "//lint:stylelint",
+)
+
 stardoc_with_diff_test(
     name = "ruff",
     bzl_library_target = "//lint:ruff",

example/.aspect/cli/config.yaml

@@ -5,5 +5,6 @@ lint:
     - //tools/lint:linters.bzl%flake8
     - //tools/lint:linters.bzl%ktlint
     - //tools/lint:linters.bzl%pmd
+    - //tools/lint:linters.bzl%stylelint
     - //tools/lint:linters.bzl%ruff
     - //tools/lint:linters.bzl%vale

example/BUILD.bazel

@@ -36,6 +36,11 @@ js_library(
     ],
 )
 
+js_library(
+    name = "stylelintrc",
+    srcs = ["stylelint.config.mjs"],
+)
+
 # NB: this alias does NOT cause Bazel's Loading phase to load the tools/BUILD file.
 # That's important as we don't want users to wait for "Eager fetching" for ~EVERY language which
 # that build file loads from.

example/lint.sh

@@ -37,7 +37,7 @@ if [ $machine == "Windows" ]; then
 	# avoid missing linters on windows platform
 	args=("--aspects=$(echo //tools/lint:linters.bzl%{flake8,pmd,ruff,vale,clang_tidy} | tr ' ' ',')")
 else
-	args=("--aspects=$(echo //tools/lint:linters.bzl%{buf,eslint,flake8,ktlint,pmd,ruff,shellcheck,vale,clang_tidy} | tr ' ' ',')")
+	args=("--aspects=$(echo //tools/lint:linters.bzl%{buf,eslint,flake8,ktlint,pmd,ruff,shellcheck,stylelint,vale,clang_tidy} | tr ' ' ',')")
 fi
 
 # NB: perhaps --remote_download_toplevel is needed as well with remote execution?

example/package.json

@@ -9,6 +9,7 @@
     "eslint": "^9",
     "prettier": "^2.8.7",
     "prettier-plugin-sql": "^0.14.0",
+    "stylelint": "^16",
     "typescript": "4.9.5",
     "typescript-eslint": "^7.10.0"
   },