Skip to content

Fix the Sync PSGalleryModules to Artifacts build#12277

Merged
adityapatwardhan merged 2 commits intoPowerShell:masterfrom
daxian-dbw:sync-build
Apr 8, 2020
Merged

Fix the Sync PSGalleryModules to Artifacts build#12277
adityapatwardhan merged 2 commits intoPowerShell:masterfrom
daxian-dbw:sync-build

Conversation

@daxian-dbw
Copy link
Copy Markdown
Member

@daxian-dbw daxian-dbw commented Apr 7, 2020

PR Summary

Fix the Sync PSGalleryModules to Artifacts build that has been failing since 3/8
The fix has been validated by the build. You can find the successful build at https://dev.azure.com/mscodehub/PowerShellCore/_build/results?buildId=131454&view=results

It's easier to review this PR when ignoring all whitespace character changes:
https://github.com/PowerShell/PowerShell/pull/12277/files?w=1

PR Checklist

@ghost ghost assigned adityapatwardhan Apr 7, 2020
@daxian-dbw daxian-dbw added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Apr 7, 2020
@daxian-dbw daxian-dbw added this to the 7.1.0-preview.2 milestone Apr 7, 2020
@rjmholt
Copy link
Copy Markdown
Collaborator

rjmholt commented Apr 7, 2020

I think given the fact that this CI failure is now occurring everywhere, it's reasonable not to block this PR on it

@daxian-dbw
Copy link
Copy Markdown
Member Author

@PoshChan Please retry macos

@PoshChan
Copy link
Copy Markdown
Collaborator

PoshChan commented Apr 7, 2020

@daxian-dbw, successfully started retry of PowerShell-CI-macOS

@adityapatwardhan adityapatwardhan merged commit eb3e4a8 into PowerShell:master Apr 8, 2020
@daxian-dbw daxian-dbw deleted the sync-build branch April 8, 2020 00:42
@ghost
Copy link
Copy Markdown

ghost commented Apr 23, 2020

🎉v7.1.0-preview.2 has been released which incorporates this pull request.:tada:

Handy links:

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.

4 participants