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

ci: add Ruby 3 to Github Actions and Appveyor #167

Merged
merged 2 commits into from
Feb 26, 2021

Conversation

flavorjones
Copy link
Collaborator

  • remove pre-2.5 from Appveyor
  • drop travis entirely

Supersedes #139

@flavorjones flavorjones force-pushed the flavorjones-update-ci-ruby-versions branch from 136ae20 to d32e3e9 Compare February 26, 2021 19:14
@flavorjones
Copy link
Collaborator Author

@stevecheckoway @rubys How do y'all feel about me removing travis in this commit? I'm confident that the current Github Actions entirely reproduce, and are faster than, the Travis config.

@stevecheckoway
Copy link
Collaborator

Thank you for doing this! I originally asked @rubys to add Appveyor because Travis didn't support Windows. Now that GitHub Actions supports Windows, is there any reason to keep Appveyor? It seems to be much slower.

@flavorjones flavorjones force-pushed the flavorjones-update-ci-ruby-versions branch from d32e3e9 to 5ca85b8 Compare February 26, 2021 20:01
@flavorjones flavorjones force-pushed the flavorjones-update-ci-ruby-versions branch from 5ca85b8 to 8f00905 Compare February 26, 2021 20:05
@flavorjones
Copy link
Collaborator Author

@stevecheckoway That's a good question, and I think the answer is probably "No." I'll probably put removal of appveyor in a separate PR just to make sure we get a solid "green" across both GA and Appveyor, which we haven't had.

also:
- Appveyor:
  - move to Visual Studio 2019 for Ruby 2.7 and 3.0
  - remove Ruby 2.4 and earlier
- Travis: drop entirely
@flavorjones flavorjones force-pushed the flavorjones-update-ci-ruby-versions branch from 288b3da to f1ca1b6 Compare February 26, 2021 20:17
@flavorjones flavorjones merged commit 0049b89 into master Feb 26, 2021
@flavorjones flavorjones deleted the flavorjones-update-ci-ruby-versions branch February 26, 2021 21:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants