Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update GH Test Matrix with minimum & latest JRuby version #197

Merged
merged 1 commit into from
Jan 18, 2021
Merged

Update GH Test Matrix with minimum & latest JRuby version #197

merged 1 commit into from
Jan 18, 2021

Conversation

tedt10e
Copy link
Contributor

@tedt10e tedt10e commented Jan 15, 2021

No description provided.

@tedt10e
Copy link
Contributor Author

tedt10e commented Jan 15, 2021

There seems to be an issue with the bundler version at 'appraisal install' in Github action. I'm not sure how to fix it.

@bensheldon
Copy link
Owner

@tedhexaflow Thanks for opening this! I fixed the bundler issue on main. If you rebase this should pass.

I'd like it if this PR bracketed the JRuby versions: have one value in the matrix for the minimum version of JRuby that GoodJob supports, and one value in the matrix for the latest version of JRuby.

@tedt10e tedt10e changed the title Update GH Test Matrix with latest JRuby version Update GH Test Matrix with minimum & latest JRuby version Jan 16, 2021
@tedt10e
Copy link
Contributor Author

tedt10e commented Jan 16, 2021

@bensheldon Rebased and updated. All checks have passed too.

Please let me know if there's anything else I need to do.

Many Thanks 💚💙💜

@bensheldon
Copy link
Owner

@tedhexaflow Awesome! Please squash this down to a single commit and I'll merge. Thank you!

@tedt10e
Copy link
Contributor Author

tedt10e commented Jan 18, 2021

@bensheldon Sorry about that. Just squashed and pushed. Thanks 💛💚❤️

@bensheldon bensheldon merged commit f37b292 into bensheldon:main Jan 18, 2021
@tedt10e tedt10e deleted the gh-test-matrix-jruby-9.2.14.0 branch January 18, 2021 04:10
@bensheldon bensheldon added the dependencies Pull requests that update a dependency file label Feb 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants