8000 Fix dependencies of nuget build to wait on DEB uploads to finish by adityapatwardhan · Pull Request #9118 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]

Skip to content

Fix dependencies of nuget build to wait on DEB uploads to finish#9118

Merged
TravisEz13 merged 1 commit intoPowerShell:masterfrom
adityapatwardhan:FixNugetArm32
Mar 11, 2019
Merged

Fix dependencies of nuget build to wait on DEB uploads to finish#9118
TravisEz13 merged 1 commit intoPowerShell:masterfrom
adityapatwardhan:FixNugetArm32

Conversation

@adityapatwardhan
Copy link
Member
@adityapatwardhan adityapatwardhan commented Mar 11, 2019

PR Summary

The NuGet build task should wait on uploads for DEB job to complete.

PR Context

There is a race condition when the build for DEB is completed but the upload has not. If the NuGet build job starts it may not get all the packages needed for building nuget packages.

PR Checklist

Copy link
Member
@daxian-dbw daxian-dbw left a comment

Choose a reason for hiding this comment

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

LGTM

@daxian-dbw daxian-dbw added this to the 6.2.0 milestone Mar 11, 2019
@TravisEz13 TravisEz13 merged commit ab1d477 into PowerShell:master Mar 11, 2019
@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Mar 11, 2019
@adityapatwardhan adityapatwardhan deleted the FixNugetArm32 branch March 11, 2019 22:09
TravisEz13 pushed a commit that referenced this pull request Mar 13, 2019
The NuGet build task should wait on uploads for DEB job to complete.

## PR Context

There is a race condition when the build for DEB is completed but the upload has not. If the NuGet build job starts it may not get all the packages needed for building nuget packages.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0