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

[Step 3] clean up #15301

Closed
wants to merge 1 commit into from
Closed

[Step 3] clean up #15301

wants to merge 1 commit into from

Conversation

ThomasCJY
Copy link
Contributor

Follow up for #15295 and Step 3 for #15176

@ThomasCJY
Copy link
Contributor Author

@coeuvre @meteorcloudy ^ ptal

@ckolli5 ckolli5 requested a review from coeuvre April 20, 2022 20:20
@ckolli5 ckolli5 added the team-Remote-Exec Issues and PRs for the Execution (Remote) team label Apr 20, 2022
coeuvre
coeuvre approved these changes Apr 21, 2022
View reviewed changes
Copy link
Member

@coeuvre coeuvre left a comment

Choose a reason for hiding this comment

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

Thanks!

@meteorcloudy Can you help import this change?

@bazel-io bazel-io closed this in ca40a51 Apr 21, 2022
coeuvre pushed a commit to coeuvre/bazel that referenced this pull request May 2, 2022
@ThomasCJY @coeuvre
clean up
4f3fac7 
Partial commit for third_party/*, see bazelbuild#15301.

Closes: bazelbuild#15301

Signed-off-by: Yun Peng <pcloudy@google.com>
ckolli5 pushed a commit that referenced this pull request May 9, 2022
@coeuvre @jchen-sc @ThomasCJY
[5.2] Upgrade Google Auth Version (#15383)
f0213bb 
* [Step 1] Upgrade Google Auth Version

third party only changes

gson

add back jackson

Partial commit for third_party/*, see #15215.

Closes #15215

Signed-off-by: Yun Peng <pcloudy@google.com>

* [Step 2] Update non-third party change for jackson2

Follow up for #15215 and Step two for #15176

Closes #15295.

PiperOrigin-RevId: 443050332

* clean up

Partial commit for third_party/*, see #15301.

Closes: #15301

Signed-off-by: Yun Peng <pcloudy@google.com>

Co-authored-by: Thomas Chen <jchen@snapchat.com>
Co-authored-by: ThomasCJY <thomascjy93@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coeuvre coeuvre coeuvre approved these changes

Assignees
No one assigned
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ThomasCJY @coeuvre @ckolli5