Skip to content

Move build to GitHub Actions#5338

Merged
carlosantoniodasilva merged 8 commits into
masterfrom
ca-build
Feb 1, 2021
Merged

Move build to GitHub Actions#5338
carlosantoniodasilva merged 8 commits into
masterfrom
ca-build

Conversation

@carlosantoniodasilva
Copy link
Copy Markdown
Member

@carlosantoniodasilva carlosantoniodasilva commented Feb 1, 2021

  • Moving all heartcombo libraries to use GA to run the build/tests.
  • Remove the gemfiles/*.lock files to keep devise consistent with the other hearcombo libs.

I had to lock devise on OmniAuth 1.0 for now otherwise some of the tests would 💥 due to a hard check we have. I'm updating it to allow 2.0 on #5327, I'll catch up that branch once this gets merged.

Keep only the lock for the main Gemfile to keep Devise consistent with
the other heartcombo libs.
Bundler 1.x uses git instead of https by default and we don't have that
github source setting in the Gemfile, but this should work.
ruby-head was always in allowed failures with travis anyway, and we
never really paid much attention to it, so let's just remove it entirely
for now.
The build is breaking with 2.x (which is expected), so this is a step to
get it to green on GA.
@carlosantoniodasilva carlosantoniodasilva self-assigned this Feb 1, 2021
@carlosantoniodasilva carlosantoniodasilva marked this pull request as ready for review February 1, 2021 14:49
@carlosantoniodasilva carlosantoniodasilva merged commit aa8d895 into master Feb 1, 2021
@carlosantoniodasilva carlosantoniodasilva deleted the ca-build branch February 1, 2021 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant