-
Notifications
You must be signed in to change notification settings - Fork 541
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
feat: Add support for python-wheel data directory #1801
feat: Add support for python-wheel data directory #1801
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
a4d20e5
to
9469321
Compare
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.
It would be also great to add an example in examples/wheel
directory.
I think the implementation is really nice, thank you a lot for submitting this.
5664bf2
to
b7c6da6
Compare
OK, I think this PR is ready. I updated the code, added a data check, added another test and an example. @rickeylev, I think this needs your approval. |
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.
I can also review, stamp and merge this :)
b7c6da6
to
3b5dc2c
Compare
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.
One more comment, but looksgood otherwise
This feature addes attribute to the py_wheel rule and makes a related change to the wheelmaker tool. Fixes bazelbuild#1777
3b5dc2c
to
2090a16
Compare
Fixes #1777
data_files
attribute topy_wheel
rule.