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.

Sign up for GitHub

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

Jump to bottom

Fix deps target #17199

Closed
wants to merge 2 commits into from
Closed

Fix deps target #17199

wants to merge 2 commits into from

Conversation

vbhavsar
Copy link

Fixes #17198

@vbhavsar vbhavsar requested a review from comius as a code owner January 12, 2023 12:31
@google-cla
Copy link

google-cla bot commented Jan 12, 2023

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.

@sgowroji sgowroji added team-Documentation Documentation improvements that cannot be directly linked to other team labels awaiting-user-response Awaiting a response from the author labels Jan 12, 2023
@github-actions
Copy link

Thank you for contributing to the Bazel repository! This pull request has been marked as stale since it has not had any activity in the last 30 days. It will be closed in the next 14 days unless any other activity occurs or one of the following labels is added: "not stale", "awaiting-review", "awaiting-PR-merge". Please provide a status update or let us know when you're ready.

@github-actions github-actions bot added the stale Issues or PRs that are stale (no activity for 30 days) label Feb 16, 2023
@vbhavsar
Copy link
Author

CLA check now passes. Can the checks be re-triggered?

@github-actions github-actions bot removed the stale Issues or PRs that are stale (no activity for 30 days) label Feb 17, 2023
@sgowroji
Copy link
Member

Hi @vbhavsar, I triggered the checks couple of times but still the buildkite checks are failling. Could you please have a look and fix the above errors, So that we can share it for further review. Thanks!

@keertk keertk added team-Rules-Python Native rules for Python and removed team-Documentation Documentation improvements that cannot be directly linked to other team labels labels Mar 28, 2023
@vbhavsar @rickeylev
Update src/main/java/com/google/devtools/build/lib/bazel/rules/python…
9da7601 
…/BazelPyBinaryRule.java

Co-authored-by: Richard Levasseur <richardlev@gmail.com>
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 1, 2025

Thank you for contributing to the Bazel repository! This pull request has been marked as stale since it has not had any activity in the last 1+ years. It will be closed in the next 90 days unless any other activity occurs. If you think this PR is still relevant and should stay open, please post any comment here and the PR will no longer be marked as stale.

@github-actions github-actions bot added the stale Issues or PRs that are stale (no activity for 30 days) label Feb 1, 2025
@vbhavsar
Copy link
Author

vbhavsar commented Feb 1, 2025

Not sure why this isn't being reviewed. Closing.

@vbhavsar vbhavsar closed this Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rickeylev rickeylev rickeylev requested changes

@comius comius Awaiting requested review from comius comius is a code owner

Assignees
No one assigned
Labels
awaiting-user-response Awaiting a response from the author stale Issues or PRs that are stale (no activity for 30 days) team-Rules-Python Native rules for Python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bazel.build] Problem with /reference/be/python
4 participants
@vbhavsar @sgowroji @rickeylev @keertk