[Bug]: lint_stylelint_aspect does not include config dependencies #394
Labels
bug
Something isn't working
Comments
sallustfire
added a commit
to sallustfire/rules_lint
that referenced
this issue
Sep 20, 2024
If the stylelint config is declared via js_library and has a dependency on other sources or npm packages, these dependencies are now included in the linting action.
alexeagle
pushed a commit
that referenced
this issue
Sep 20, 2024
* fix: include stylelint config deps (#394) If the stylelint config is declared via js_library and has a dependency on other sources or npm packages, these dependencies are now included in the linting action. * fix bzl_library deps * update docs * format config file
|
fixed by #395 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What happened?
In analogy with eslint, the typical setup for the stylelint config file is to copy it to the output tree as a js_library. The config file may depend on external stylelint plugins which need to be made available via runfiles. The implementation only declares a dependency on the config file and does not include the config file's dependencies.
Version
Development (host) and target OS/architectures: Pop!_OS 22.04 LTS
Output of
bazel --version: 7.1.1Language(s) and/or frameworks involved: stylelint
How to reproduce
Any other information?
No response
The text was updated successfully, but these errors were encountered: