Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Remove Appveyor #613

Closed
IllusionMH opened this issue Oct 29, 2018 · 2 comments
Closed

Remove Appveyor #613

IllusionMH opened this issue Oct 29, 2018 · 2 comments
Labels
Domain: Tooling Repository tasks around improving source tooling.
Milestone
6.0.0

Comments

@IllusionMH
Copy link
Contributor

After #596 was merged - now Travis CI verifies on both Linux and Windows.

The question is: is there a need for two CI tools?
@JoshuaKGoldberg
Copy link

Great observation - no, and anecdotally I found Appveyor to be slower than Travis. We can probably get rid of it, but I'll try to get input from @HamletDRC first.

@JoshuaKGoldberg JoshuaKGoldberg added Type: Question Domain: Tooling Repository tasks around improving source tooling. Status: Awaiting More Feedback labels Oct 30, 2018
@HamletDRC
Copy link
Member

@JoshuaKGoldberg sure, get rid of Appveyor.

@mesaugat mesaugat mentioned this issue Nov 1, 2018
Closed
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Nov 5, 2018
Merged
@mesaugat mesaugat mentioned this issue Nov 5, 2018
Merged
@JoshuaKGoldberg JoshuaKGoldberg changed the title Question: Is Appveyor still required? Remove Appveyor Nov 5, 2018
@IllusionMH IllusionMH added this to the 6.0.0 milestone Feb 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Domain: Tooling Repository tasks around improving source tooling.
Projects
None yet
Milestone
6.0.0
Development

No branches or pull requests
3 participants
@HamletDRC @IllusionMH @JoshuaKGoldberg