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

Convert from Travis CI to GitHub Actions #130

Merged
merged 3 commits into from
Apr 19, 2022
Merged

Convert from Travis CI to GitHub Actions #130

merged 3 commits into from
Apr 19, 2022

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Mar 11, 2022

This is still a draft because I need to figure out the test matrix. It should also add a release workflow.

@ekohl ekohl force-pushed the gha branch 4 times, most recently from f986fc9 to aa30541 Compare March 11, 2022 11:14
@ekohl
Copy link
Member Author

ekohl commented Mar 11, 2022

I do not know why the tests are failing. They pass locally for me.

@ekohl ekohl mentioned this pull request Mar 11, 2022
GitHub action install the bundle in the vendor directory in the project
root.  We want to ignore it.
@smortex
Copy link
Member

smortex commented Apr 19, 2022

I could reproduce the failure by running bundle --path vendor/foo --without development release on my system running Ruby 2.7.

The remaining errors with Ruby 3.1 seems related to #129

@ekohl
Copy link
Member Author

ekohl commented Apr 19, 2022

Looks like the patch in incompatible with Ruby < 2.6

Ruby 3.1 deprecates the old style, but Ruby < 2.6 doesn't accept the new
style. This is compatible with both.
@ekohl ekohl marked this pull request as ready for review April 19, 2022 11:13
@ekohl
Copy link
Member Author

ekohl commented Apr 19, 2022

Now it's green.

@ekohl ekohl added the enhancement New feature or request label Apr 19, 2022
@ekohl ekohl merged commit 5295d98 into voxpupuli:master Apr 19, 2022
@ekohl ekohl deleted the gha branch April 19, 2022 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants