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(gazelle): generate empty py_library only when need #1905

Merged
merged 2 commits into from
May 16, 2024

Conversation

hunshcn
Copy link
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
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
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 bazelbuild:main with commit 3744e4d May 16, 2024
4 checks passed
@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