Skip to content

Publish test and code coverage artifacts for daily builds - #8955

Merged
adityapatwardhan merged 2 commits into
PowerShell:masterfrom
adityapatwardhan:FixDailyTestPackageWindows
Feb 25, 2019
Merged

Publish test and code coverage artifacts for daily builds#8955
adityapatwardhan merged 2 commits into
PowerShell:masterfrom
adityapatwardhan:FixDailyTestPackageWindows

Conversation

@adityapatwardhan

@adityapatwardhan adityapatwardhan commented Feb 23, 2019

Copy link
Copy Markdown
Member

PR Summary

Reenable publishing code coverage and test artifacts

PR Context

Refactoring AppVeyor.psm1 into ci.psm1 clobbered the change for publishing code coverage and test artifacts on Windows.

PR Checklist

@RDIL

RDIL commented Feb 25, 2019

Copy link
Copy Markdown
Contributor

@adityapatwardhan can you please add fixes #8930 to the body of the PR so it closes the issue? All looks good so far!

@adityapatwardhan

Copy link
Copy Markdown
Member Author

@RDIL it is related but does not completely fix #8930

@RDIL

RDIL commented Feb 25, 2019

Copy link
Copy Markdown
Contributor

Okay.

@adityapatwardhan
adityapatwardhan force-pushed the FixDailyTestPackageWindows branch from 949e730 to affc10e Compare February 25, 2019 18:18
@adityapatwardhan
adityapatwardhan marked this pull request as ready for review February 25, 2019 18:18
@adityapatwardhan
adityapatwardhan merged commit 76e8091 into PowerShell:master Feb 25, 2019
@adityapatwardhan
adityapatwardhan deleted the FixDailyTestPackageWindows branch February 25, 2019 21:03
@TravisEz13 TravisEz13 added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 27, 2019
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

3 participants