Skip to content
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

Fix tools/BUILD.bazel to make 'bazel cquery ...' work #233

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

moroten
Copy link
Contributor

@moroten moroten commented Dec 20, 2024

ibazel (https://github.com/bazelbuild/bazel-watcher/) is running bazel cquery ... which failed when there are executable targets in the deps attribute of a go_library. This was never checked in CI because of the manual tag of the tools target.

ibazel (https://github.com/bazelbuild/bazel-watcher/) is running
`bazel cquery ...` which failed when there are executable targets in the
deps attribute of a go_library. This was never checked in CI because of
the manual tag of the tools target.
@EdSchouten EdSchouten merged commit ff8c80c into buildbarn:master Dec 20, 2024
1 check passed
@moroten moroten deleted the fix-cquery-all branch December 20, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants