fix(gazelle) Delete python targets with invalid srcs by yushan26 · Pull Request #3046 · bazel-contrib/rules_python

linzhp

dougthor42

dougthor42

linzhp

linzhp

dougthor42

dougthor42

dougthor42

@yushan8

Update with invalid sources

Update

Update with py_binary kind only

address comments

update

update test

address comments

Update PR

update with changelog

Update

Update

Address comments

Update

update

@dougthor42

@dougthor42

This was referenced

Oct 21, 2025

dougthor42 added a commit to dougthor42/rules_python that referenced this pull request

Jan 14, 2026
I've had to reference this a couple times now and keep forgetting that
it's not in the changelog haha. I figured I'd remedy that.

github-merge-queue bot pushed a commit that referenced this pull request

Jan 16, 2026
I've had to reference this a couple times now and keep forgetting that
it's not in the changelog haha. I figured I'd remedy that.

dougthor42 added a commit to dougthor42/rules_python that referenced this pull request

Feb 16, 2026
These changes will make a little more sense in a followup PR that
finishes the "delete invalid targets" work started in bazel-contrib#3046. In that PR,
I pushed back on changes that resulted in py_test and py_library targets
getting removed if they were invalid (see
bazel-contrib#3046 (review)
and
bazel-contrib#3046 (comment)).

This PR updates tests by adding some always-valid targets and does some
light renaming of the targets.

github-merge-queue bot pushed a commit that referenced this pull request

Feb 18, 2026
…3601)

Update `remove_invalid_(binary|library)` gazelle tests.

These changes will make a little more sense in a followup PR that
finishes the "delete invalid targets" work started in #3046. In that PR,
I pushed back on changes that would mean `py_test` and `py_library`
targets got removed if they were invalid (see
#3046 (review)).

It's now time to fix that and make it so _any_ invalid target, not just
`py_binary`, is removed see
#3046 (comment)).
That change will come in a followup PR.

This PR updates tests by adding some always-valid targets and does some
light renaming of the targets.