Skip to content

Fix daily CI builds to publish tar package as artifacts #8775

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.

Sign up for GitHub

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

Jump to bottom
Merged
TravisEz13 merged 2 commits into from
Feb 4, 2019
Merged

Fix daily CI builds to publish tar package as artifacts #8775

TravisEz13 merged 2 commits into from
Feb 4, 2019

Conversation

@adityapatwardhan
Copy link
Member

@adityapatwardhan adityapatwardhan commented Jan 29, 2019 •
edited
Loading

PR Summary

Update the packaging script to always create the tar.gz packages.

PR Context

Daily builds for Linux and MacOS do not publish tar.gz package.

PR Checklist

@adityapatwardhan
Publish tar.gz and TestPackage.zip on daily build
f71b555 
* Also restructured Windows daily build artifacts
* Enabled publishing artifacts on PRs.
@adityapatwardhan adityapatwardhan changed the title WIP: Fix daily CI builds to publish tar package as artifacts Fix daily CI builds to publish tar package as artifacts Feb 1, 2019
@adityapatwardhan
Copy link
Member Author

adityapatwardhan commented Feb 4, 2019

@TravisEz13 This is ready for review and merge.

@adityapatwardhan adityapatwardhan mentioned this pull request Feb 4, 2019
Merged
11 tasks
TravisEz13
TravisEz13 approved these changes Feb 4, 2019
View reviewed changes
@TravisEz13 TravisEz13 self-assigned this Feb 4, 2019
@TravisEz13 TravisEz13 merged commit ec12f37 into PowerShell:master Feb 4, 2019
@adityapatwardhan adityapatwardhan deleted the FixDailyBuildArtifacts branch February 4, 2019 22:55
@iSazonov iSazonov added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TravisEz13 TravisEz13 TravisEz13 approved these changes

@anmenaga anmenaga Awaiting requested review from anmenaga

@daxian-dbw daxian-dbw Awaiting requested review from daxian-dbw

+1 more reviewer

@PaulHigin PaulHigin PaulHigin approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@TravisEz13 TravisEz13

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@adityapatwardhan @TravisEz13 @PaulHigin @iSazonov