Skip to content

fix(gazelle): generate empty py_library only when need#1905

Merged
aignas merged 2 commits intobazel-contrib:mainfrom
hunshcn:fix/invalid-library
May 16, 2024
Merged

fix(gazelle): generate empty py_library only when need#1905
aignas merged 2 commits intobazel-contrib:mainfrom
hunshcn:fix/invalid-library

Conversation

@hunshcn
Copy link
Copy Markdown
Contributor

@hunshcn hunshcn commented May 16, 2024

#1887 incorrectly generated an empty py_library for all dirs, although it will eventually be removed because it is empty, but it will easily conflict with other existing rules.

Fix this error, and only generate an empty py_library for bazel packages with the same name py_library rule.

@hunshcn hunshcn requested a review from f0rmiga as a code owner May 16, 2024 14:14
@hunshcn
Copy link
Copy Markdown
Contributor Author

hunshcn commented May 16, 2024

The first commit is the badcase of #1887 error. The second commit was fixed.

@aignas Need to merge as soon as possible.

@hunshcn hunshcn changed the title fix/invalid library fix(gazelle): fix #1887 May 16, 2024
@hunshcn
Copy link
Copy Markdown
Contributor Author

hunshcn commented May 16, 2024

I'm sorry for this mistake.

@hunshcn hunshcn changed the title fix(gazelle): fix #1887 fix(gazelle): only generate empty py_library if need May 16, 2024
@hunshcn hunshcn changed the title fix(gazelle): only generate empty py_library if need fix(gazelle): generate empty py_library only when need May 16, 2024
@aignas aignas added this pull request to the merge queue May 16, 2024
Merged via the queue into bazel-contrib:main with commit 3744e4d May 16, 2024
@hunshcn hunshcn deleted the fix/invalid-library branch May 16, 2024 15:01
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