Skip to content

Ensure all core Python rules are loaded from defs.bzl #219

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

Merged
merged 1 commit into from
Aug 2, 2019

Conversation

brandjon
Copy link
Contributor

@brandjon brandjon commented Aug 2, 2019

This migrates rules_python itself for bazelbuild/bazel#9006. This entails
adding or updating load() statements to refer to //python:defs.bzl (not
//python:python.bzl), and updating whl.py so the generated repos refer to
@rules_python. Also updated the dependency on subpar for compatibility with
the flag. Par files are regenerated.

This migrates rules_python itself for bazelbuild/bazel#9006. This entails
adding or updating load() statements to refer to //python:defs.bzl (not
//python:python.bzl), and updating whl.py so the generated repos refer to
@rules_python. Also updated the dependency on subpar for compatibility with
the flag. Par files are regenerated.
@brandjon brandjon requested a review from gregestren August 2, 2019 19:57
@brandjon brandjon requested a review from lberki as a code owner August 2, 2019 19:57
@brandjon brandjon self-assigned this Aug 2, 2019
@brandjon brandjon merged commit 4b84ad2 into bazel-contrib:master Aug 2, 2019
@brandjon brandjon deleted the migrate-bzl branch August 2, 2019 20:01
fweikert pushed a commit to fweikert/rules_python that referenced this pull request Aug 7, 2019
)

This migrates rules_python itself for bazelbuild/bazel#9006. This entails
adding or updating load() statements to refer to //python:defs.bzl (not
//python:python.bzl), and updating whl.py so the generated repos refer to
@rules_python. Also updated the dependency on subpar for compatibility with
the flag. Par files are regenerated.
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