Skip to content

Conversation

@tvdaptible
Copy link
Contributor

@tvdaptible tvdaptible commented Dec 15, 2025

2.7 was failing due to using the older version of activesupport, which is also fixed in this pr.

also add make test RUBY_VERSION=X.Y and make lint RUBY_VERSION=X.Y to make it simple to repro issues locally

@snyk-io
Copy link

snyk-io bot commented Dec 15, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@tvdaptible tvdaptible changed the title chore: test more rubies [SC-35314] chore: test ruby 2.1 thru 3.4 [SC-35314] Dec 16, 2025
@tvdaptible tvdaptible marked this pull request as ready for review December 16, 2025 18:01
@tvdaptible tvdaptible requested a review from almathew as a code owner December 16, 2025 18:01
@tvdaptible tvdaptible changed the title chore: test ruby 2.1 thru 3.4 [SC-35314] chore: test ruby 2.3 thru 3.4 [SC-35314] Dec 16, 2025
Copy link
Member

@madhuravius madhuravius left a comment

Choose a reason for hiding this comment

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

LGTM - we MIGHT want multiple docker-compose.yml runners locally to test different versions locally but that's a huge overoptimization

@tvdaptible
Copy link
Contributor Author

tvdaptible commented Dec 16, 2025

LGTM - we MIGHT want multiple docker-compose.yml runners locally to test different versions locally but that's a huge overoptimization

what doesn't work locally?

make lint test RUBY_VERSION=2.3
make lint test RUBY_VERSION=2.4
make lint test RUBY_VERSION=2.7
make lint test RUBY_VERSION=3.3
make lint test RUBY_VERSION=3.4

@tvdaptible tvdaptible merged commit 7049d7d into master Dec 16, 2025
15 checks passed
@tvdaptible tvdaptible deleted the no-more-ruby-21-22 branch December 16, 2025 18:39
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.

4 participants