Skip to content

fix: give build_test a descriptive rule kind - #636

Draft
tomatotomata wants to merge 1 commit into
bazelbuild:mainfrom
tomatotomata:codex/skylib-build-test-kind-602
Draft

fix: give build_test a descriptive rule kind#636
tomatotomata wants to merge 1 commit into
bazelbuild:mainfrom
tomatotomata:codex/skylib-build-test-kind-602

Conversation

@tomatotomata

Copy link
Copy Markdown

Summary

  • rename the internal rule kind generated by build_test from _empty_test to skylib_build_test
  • keep the public build_test macro and generated target names unchanged

This addresses #602 and makes bazel query 'kind(skylib_build_test, //...)' readable without changing the macro API.

Validation

  • git diff --check
  • existing //tests:build_test_test coverage remains the focused test target

Bazel 8.x is not installed in my Windows checkout, so I could not run the Bazel test locally. The commit includes a DCO sign-off.

Signed-off-by: ahmadalguydi <ahmadalgaidy@hotmail.com>
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.

1 participant