Skip to content

Publish test package to AppVeyor daily build #8273

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
adityapatwardhan merged 3 commits into from
Nov 26, 2018
Merged

Publish test package to AppVeyor daily build #8273

adityapatwardhan merged 3 commits into from
Nov 26, 2018

Conversation

@adityapatwardhan
Copy link
Member

@adityapatwardhan adityapatwardhan commented Nov 14, 2018 •
edited
Loading

PR Summary

PR #8220 inadvertently reverted the work done in #8087

This PR brings the code back.

PR Checklist

@iSazonov iSazonov added the CL-Test Indicates that a PR should be marked as a test change in the Change Log label Nov 16, 2018
@TravisEz13
Copy link
Member

TravisEz13 commented Nov 16, 2018

@adityapatwardhan there are test failures

@anmenaga
Copy link

anmenaga commented Nov 20, 2018

On PowerShell-CI-windows:
Invoke-AppveyorAfterTest : The term 'Push-AppveyorArtifact' is not recognized as the name of a cmdlet, function, script file, or operable program.

@adityapatwardhan
Copy link
Member Author

adityapatwardhan commented Nov 20, 2018

@anmenaga Please have a look. There was a typo causing the issue.

@adityapatwardhan adityapatwardhan merged commit 9fda1d0 into PowerShell:master Nov 26, 2018
@adityapatwardhan adityapatwardhan added CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log and removed CL-Test Indicates that a PR should be marked as a test change in the Change Log labels Nov 26, 2018
@adityapatwardhan adityapatwardhan deleted the TestPackage branch November 26, 2018 18:46
iSazonov pushed a commit to iSazonov/PowerShell that referenced this pull request Nov 29, 2018
@adityapatwardhan @iSazonov
Publish test package to AppVeyor daily build (PowerShell#8273)
a3508cc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TravisEz13 TravisEz13 Awaiting requested review from TravisEz13

@PaulHigin PaulHigin Awaiting requested review from PaulHigin

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

1 more reviewer

@anmenaga anmenaga anmenaga approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

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 @anmenaga @iSazonov