Skip to content
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.

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

Remove appveyor functions from New-DockerTestBuild.psm1 #8756

Merged
merged 3 commits into from
Jan 31, 2019
Merged

Remove appveyor functions from New-DockerTestBuild.psm1 #8756

merged 3 commits into from
Jan 31, 2019

Conversation

RDIL
Copy link
Contributor

@RDIL RDIL commented Jan 27, 2019

PR Summary

Remove interactions to appveyor api in this file

PR Context

#8686

PR Checklist

@TravisEz13
Copy link
Member

@JamesWTruher Do you want to do anything with this code before it is removed?

@JamesWTruher
Copy link
Member

@TravisEz13 nothing is ever gone in github :) if we're not using it, we should pull it. I can always get it if the need arises (and start using AppVeyor again)

@anmenaga anmenaga merged commit 9e4e50a into PowerShell:master Jan 31, 2019
@RDIL RDIL deleted the patch-3 branch January 31, 2019 19:15
@anmenaga anmenaga added WG-Quality-Test issues in a test or in test infrastructure Area-Maintainers-Build specific to affecting the build labels Jan 31, 2019
@iSazonov iSazonov added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Maintainers-Build specific to affecting the build CL-Tools Indicates that a PR should be marked as a tools change in the Change Log WG-Quality-Test issues in a test or in test infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants