aspect-build / aspect-build/rules_lint
[Bug]: aspect_rules_lint 2.x incompatible with Bazel 8.x
- Dominant language
- Starlark
- Stars
- 154
- Forks
- 125
- Avg merge
- 3d 21h
- Merged PRs (30d)
- 20
Description
### What happened?
repo_name = None in MODULE.bazel file breaks Bazel 8.x builds although bazel_compatibility = [">=7.6.0"]
Error Log:
ERROR: .../MODULE.bazel:17:10: Traceback (most recent call last):
File ".../MODULE.bazel", line 17, column 10, in
Error in bazel_dep: in call to bazel_dep(), parameter 'repo_name' got value of type 'NoneType', want 'string'
### Version
Affected Versions:
- `aspect_rules_lint` 2.0.0, 2.1.0, and current main
- Bazel 7.6.0 - 8.x (tested on 8.1.0)
### How to reproduce
```shell
1. Create a test directory with these files:
- MODULE.bazel
module(name = "repo_name_none_bug_test", version = "0.0.1")
bazel_dep(name = "bazel_skylib", version = "1.5.0", repo_name = None)
- BUILD
2. bazel build //...
```
### Any other information?
Bazel 8.x requires `repo_name` to be a string (or omitted). However Bazel 9+ tolerates `None`.
Contributor guide
Research direction
Reproduce the report in a test directory using the shown MODULE.bazel and BUILD files, then run `bazel build //...` with Bazel 8.x. Trace the `repo_name` value back to the rules_lint configuration or entry point that generates it; done means the reproduction builds successfully while the stated Bazel compatibility remains valid.
Written by the indexing model from the issue text.
Assessment
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100