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

@ethomson ethomson commented Aug 6, 2018

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
Copy link
Copy Markdown
Member Author

ethomson commented Aug 6, 2018

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

@pks-t
Copy link
Copy Markdown
Member

pks-t commented Aug 9, 2018

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

tiennou commented Aug 9, 2018

FWIW, I'm 👍 too.

@ethomson
Copy link
Copy Markdown
Member Author

ethomson commented Aug 9, 2018

🔥

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