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

Download artifacts from partially succeeded builds #8660

Merged
merged 2 commits into from
Nov 25, 2018

Conversation

Thilas
Copy link
Contributor

@Thilas Thilas commented Oct 20, 2018

Per #8431, add option to DownloadBuildArtifactsV0 task in order to allow to download artifacts from partially succeeded builds.

@Thilas
Copy link
Contributor Author

Thilas commented Oct 20, 2018

@omeshp, please review my PR.

FYI, I'm willing to add tests if required but I would just need some directions since there are none on the DownloadBuildArtifactsV0 task for now.

@Thilas
Copy link
Contributor Author

Thilas commented Oct 20, 2018

@GitHubSriramB, @kasubram, if any of you could review this PR as well, I would be great. Thanks.

@Thilas
Copy link
Contributor Author

Thilas commented Nov 8, 2018

@kmkumaran, any news about this?

@Lovakumar Lovakumar self-requested a review November 8, 2018 06:57
@kmkumaran kmkumaran assigned Lovakumar and unassigned kmkumaran Nov 9, 2018
@kmkumaran
Copy link
Member

@Thilas - @kasubram and @omeshp will own reviewing this PR. They will respond to you on this.

Tasks/DownloadBuildArtifactsV0/task.json Outdated Show resolved Hide resolved
Thomas Démoulins added 2 commits November 9, 2018 20:26
@Thilas Thilas force-pushed the improve-download-artifact branch from 72a8881 to 5d340f7 Compare November 9, 2018 19:30
@Thilas
Copy link
Contributor Author

Thilas commented Nov 25, 2018

What's next? It has been approved so I guess it just need to be merged now, right? Thanks.

@kasubram kasubram merged commit f8526d8 into microsoft:master Nov 25, 2018
@Thilas Thilas deleted the improve-download-artifact branch December 6, 2018 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants