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
[Bazel] Move Python rules to //python #9952
Conversation
dlj-NaN
commented
May 12, 2022
- This PR moves Python rules to //python, but leaves aliases for publicly-visible targets.
- There is some internal cleanup for how we copy .proto sources so they are usable from Python.
- python/BUILD.bazel is now more in line with Bazel's BUILD style guide.
c2f1c61
to
25df3da
Compare
DefaultInfo(files = depset([dest for src, dest in src_dests])), | ||
] | ||
|
||
internal_copy_files_impl = rule( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add some more comments for the semantics of this rule and the attributes?
It's not immediately clear to me what this rule does at a high level, or why it needs to make a batch file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.