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

Add digest func to Remote Asset Downloader #21996

Closed

Conversation

sluongng
Copy link
Contributor

@sluongng sluongng commented Apr 15, 2024

A follow up from bazelbuild/remote-apis#286

@sluongng sluongng marked this pull request as ready for review April 15, 2024 10:23
@sluongng sluongng requested a review from a team as a code owner April 15, 2024 10:23
@github-actions github-actions bot added team-Remote-Exec Issues and PRs for the Execution (Remote) team awaiting-review PR is awaiting review from an assigned reviewer labels Apr 15, 2024
@coeuvre coeuvre added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 16, 2024
@sluongng
Copy link
Contributor Author

@coeuvre What do you think about including this in 7.2.0?

copybara-service bot pushed a commit that referenced this pull request Apr 17, 2024
Partial commit for third_party/*, see #21996.

Change-Id: I3acfc5dc7b0fb118ace627721830dc657c80033f
Signed-off-by: iancha1992 <heec@google.com>
@coeuvre
Copy link
Member

coeuvre commented Apr 18, 2024

@coeuvre What do you think about including this in 7.2.0?

Let's do it!

@coeuvre
Copy link
Member

coeuvre commented Apr 18, 2024

@bazel-io fork 7.2.0

bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Apr 25, 2024
A follow up from bazelbuild/remote-apis#286

Closes bazelbuild#21996.

PiperOrigin-RevId: 628100606
Change-Id: Ib37a013d704dcfd14556f7914d90c1393ef73d38
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 26, 2024
iancha1992 pushed a commit to bazel-io/bazel that referenced this pull request Apr 26, 2024
Partial commit for third_party/*, see bazelbuild#21996.

Change-Id: I3acfc5dc7b0fb118ace627721830dc657c80033f
Signed-off-by: iancha1992 <heec@google.com>
github-merge-queue bot pushed a commit that referenced this pull request Apr 29, 2024
A follow up from bazelbuild/remote-apis#286

Closes #21996.

PiperOrigin-RevId: 628100606
Change-Id: Ib37a013d704dcfd14556f7914d90c1393ef73d38

Commit
618c0ab
Commit
dcfefd9
(Third party changes)

---------

Signed-off-by: iancha1992 <heec@google.com>
Co-authored-by: Son Luong Ngoc <sluongng@gmail.com>
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
Partial commit for third_party/*, see bazelbuild#21996.

Change-Id: I3acfc5dc7b0fb118ace627721830dc657c80033f
Signed-off-by: iancha1992 <heec@google.com>
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
A follow up from bazelbuild/remote-apis#286

Closes bazelbuild#21996.

PiperOrigin-RevId: 628100606
Change-Id: Ib37a013d704dcfd14556f7914d90c1393ef73d38
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 7.2.0 RC1. Please test out the release candidate and report any issues as soon as possible.
If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=7.2.0rc1. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Remote-Exec Issues and PRs for the Execution (Remote) team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants