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

Use newer Starlark APIs #2

Merged
merged 1 commit into from
Mar 27, 2019
Merged

Use newer Starlark APIs #2

merged 1 commit into from
Mar 27, 2019

Conversation

drigz
Copy link
Contributor

@drigz drigz commented Feb 21, 2019

single_file will be replaced by allow_single_file.
ctx.action will be replaced by ctx.actions.run and
ctx.actions.run_shell.

single_file will be replaced by allow_single_file.
ctx.action will be replaced by ctx.actions.run and
ctx.actions.run_shell.
cloud-robotics-github-robot pushed a commit to googlecloudrobotics/core that referenced this pull request Feb 21, 2019
This addresses use of single_file, and pulls in some changes to
rules_python from upstream. This is a contribution to
#2.

I have also sent a pull request for bazel_rules:
antonovvk/bazel_rules#2

There are many other failures, including:

- java_grpc_library.bzl from grpc/grpc-java
- protobuf.bzl from google/protobuf
- rules_go
- references to genfiles (this will be unified with `bin`).

Change-Id: Ic6548244f0906cdbd53aed16017191b6db8cd553
GitOrigin-RevId: daa70bc
Copy link
Owner

@antonovvk antonovvk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for contributing!

@cschuet
Copy link

cschuet commented Mar 27, 2019

Can you merge this please?

@antonovvk antonovvk merged commit b3813bf into antonovvk:master Mar 27, 2019
@antonovvk
Copy link
Owner

Oops, sorry, I thought the author would do it after approval and forgot to check it out.

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.

3 participants