Skip to content

ci: remove appveyor#4760

Merged
ethomson merged 2 commits into
masterfrom
ethomson/ci
Aug 9, 2018
Merged

ci: remove appveyor#4760
ethomson merged 2 commits into
masterfrom
ethomson/ci

Conversation

@ethomson

@ethomson ethomson commented Aug 6, 2018

Copy link
Copy Markdown
Member

Remove AppVeyor. It's a fine provider, but unfortunately builds queue and complete very slowly and on top of that, I'm paying cash money for it. After adding VSTS builds, they're not providing additional benefits to the PR or nightly build setup.

@ethomson

ethomson commented Aug 6, 2018

Copy link
Copy Markdown
Member Author

(This PR is rather symbolic - I'll detach the AppVeyor configuration once we agree that this is good and right.)

@pks-t

pks-t commented Aug 9, 2018

Copy link
Copy Markdown
Member

AppVeyor is horribly slow, and VSTS is a lot quicker even though it also builds Linux and macOS targets. So I'm all for removing AppVeyor

@tiennou

tiennou commented Aug 9, 2018

Copy link
Copy Markdown
Contributor

FWIW, I'm 👍 too.

@ethomson

ethomson commented Aug 9, 2018

Copy link
Copy Markdown
Member Author

🔥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants