Skip to content

Publish test and code coverage artifacts for daily builds#8955

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

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

Conversation

@adityapatwardhan
Copy link
Copy Markdown
Member

@adityapatwardhan adityapatwardhan commented Feb 23, 2019

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
Copy link
Copy Markdown
Contributor

RDIL commented Feb 25, 2019

@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
Copy link
Copy Markdown
Contributor

RDIL commented Feb 25, 2019

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants