-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: bazel lint //...
leads to No aspects enabled for linting. ...
even though config.yaml file exists
#677
Comments
Oops, I'm sorry we missed your original issue on aspect-cli repo. I'm not able to reproduce. You report that |
No worries! Hmm, yeah, that's right, do you have any ideas on how I could investigate this? It seems to happen on my teammates machines as well so I'm wondering if there are any investigation leads I could follow. |
We hit this too, looks like it has something to do with workspace detection. The workaround we used: In
Add empty |
Thank you for sharing, that did fix it! |
Could either of you share what the repository looked like before adding that workaround? From the code here https://github.com/aspect-build/aspect-cli/blob/main/pkg/bazel/workspace/finder.go#L27 I'd expect you need a |
In our repository there was a Also, found bazel-contrib/vscode-bazel#323 |
What happened?
Running
bazel lint //...
leads toRunning
bazel lint --aspect:config "$(pwd)/.aspect/cli/config.yaml" //...
from the same directory, the root of the Bazel monorepo, works.Version
Development (host) and target OS/architectures: Ubuntu 18.04
Output of
bazel --version
:aspect 5.8.20
Version of the Aspect rules, or other relevant rules from your
WORKSPACE
orMODULE.bazel
file:aspect_bazel_lib 1.38.0
,aspect_rules_lint 0.10.0
Language(s) and/or frameworks involved: not sure if relevant, the
bazel lint
command doesn't seem to properly locate the configuration file.How to reproduce
No response
Any other information?
No response
The text was updated successfully, but these errors were encountered: