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

Update README and add py_repositories() #218

Merged
merged 4 commits into from
Aug 2, 2019

Conversation

brandjon
Copy link
Member

@brandjon brandjon commented Aug 2, 2019

Update the README file to explain the difference between the core Python rules
and the packaging rules. Add the py_repositories() hook and recommend that
users add it to their WORKSPACE files.

Background on the py_repositories() hook: I don't know what this might be
useful for at the moment, but I found pip_repositories() to be helpful when I
added a deprecation message for renaming the workspace to @rules_python. So
it's probably a good idea to have this just in case.

Work toward bazelbuild/bazel#9006.

Update the README file to explain the difference between the core Python rules
and the packaging rules. Add the py_repositories() hook and recommend that
users add it to their WORKSPACE files.

Background on the py_repositories() hook: I don't know what this might be
useful for at the moment, but I found pip_repositories() to be helpful when I
added a deprecation message for renaming the workspace to @rules_python. So
it's probably a good idea to have this just in case.
@brandjon brandjon requested a review from aiuto August 2, 2019 17:28
@brandjon brandjon requested a review from lberki as a code owner August 2, 2019 17:28
@brandjon brandjon self-assigned this Aug 2, 2019
@brandjon brandjon requested a review from katre August 2, 2019 18:46
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
python/repositories.bzl Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@brandjon brandjon merged commit 6c5f479 into bazelbuild:master Aug 2, 2019
@brandjon brandjon deleted the py-repo branch August 2, 2019 19:20
fweikert pushed a commit to fweikert/rules_python that referenced this pull request Aug 7, 2019
Update the README file to explain the difference between the core Python rules
and the packaging rules. Add the py_repositories() hook and recommend that
users add it to their WORKSPACE files.

Background on the py_repositories() hook: I don't know what this might be
useful for at the moment, but I found pip_repositories() to be helpful when I
added a deprecation message for renaming the workspace to @rules_python. So
it's probably a good idea to have this just in case.

Also fix and regenerate docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants