aspect-build / aspect-build/rules_lint

[Bug]: aspect_rules_lint 2.x incompatible with Bazel 8.x

Open
#759 0 comments 2 reactions 0 assignees View on GitHub
bug
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.