-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[DownloadBuildArtifactsV0, JavaToolInstallerV0] Updated OSS dependencies #14703
[DownloadBuildArtifactsV0, JavaToolInstallerV0] Updated OSS dependencies #14703
Conversation
LGTM, but let's make buddy testing also - to double-check that it works fine. |
…vatoolinstaller-oss-update
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested these changes manually:
DownloadBuildArtifacts:
✔️Storing and not storing permissions, warnings, multiple files download and extraction, windows file share
✔️Downloading file from a different build
✔️Downloading artifact as zip when triggering build via PR from fork
JavaToolInstaller:
✔️Install from local source and azure storage
Found no issues. LGTM!
…vatoolinstaller-oss-update
…vatoolinstaller-oss-update
…vatoolinstaller-oss-update
This reverts commit 392108a.
This reverts commit 392108a.
This reverts commit 392108a.
Task name: DownloadBuildArtifactsV0, JavaToolInstallerV0
Description: Updated OSS dependencies
Documentation changes required: (Y/N) N
Added unit tests: (Y/N) N
Attached related issue: (Y/N) N
Checklist: