Skip to content

Conversation

@hsbt
Copy link
Member

@hsbt hsbt commented Jun 16, 2022

No description provided.

steps:
- id: versions
run: |
versions=$(curl -s 'https://cache.ruby-lang.org/pub/misc/ci_versions/cruby.json' | jq -c '. + ["2.4"]')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2.5, 2.6 are not in this set, should they be?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added only the minimum supported version that is "2.4" because I needed to add "2.7" into this line when Ruby 2.7 was EOL.

I'm not sure what the best way to auto-versioning with this strategy.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know either, but the tests run quickly, and perhaps there's added feedback to found in the 2.5, 2.6 test runs? Well, it's up to you.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no strong opinion that. I added them.

@hsbt hsbt merged commit 9a75e5e into master Jun 16, 2022
@hsbt hsbt deleted the ci-versions branch June 16, 2022 11:53
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