Skip to content

Try again #219

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

Merged
merged 1 commit into from
Oct 30, 2019
Merged

Try again #219

merged 1 commit into from
Oct 30, 2019

Conversation

ahayworth
Copy link
Contributor

@ahayworth ahayworth commented Oct 28, 2019

This PR successfully reworks the CI suite to run in containers on GitHub Actions.

We use a docker-based workflow to get consistent runs. Note that we use
stretch images for newer ruby versions and do some ssl hackery - more
recent debian versions see SSL test failures related to unsupported
ciphers, etc. Rather than debug them now, we ignore them - they're not
likely related directly to the code we're testing.

We also hack up the ruby docker image; because they do this strange
thing where they install gems globally into one bundle. It messes up
this build process, so we undo it to the best of our ability.
@ahayworth ahayworth marked this pull request as ready for review October 30, 2019 02:27
@ahayworth ahayworth merged commit 86db6e4 into master Oct 30, 2019
@ahayworth ahayworth deleted the ahayworth-patch-1 branch October 30, 2019 22:34
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.

1 participant