Skip to content
This repository was archived by the owner on Dec 22, 2022. It is now read-only.

Use github workflows to run CI #2125

Merged
merged 3 commits into from
Dec 23, 2020
Merged

Use github workflows to run CI #2125

merged 3 commits into from
Dec 23, 2020

Conversation

pgwillia
Copy link
Member

Context

With travis-ci transitioning to a limited credit system after Dec 31st and currently taking hours for jobs to go from queued to completion, I think we'll need to migrate all our CI to github actions

Related to #2121

What's New

  • push workflow for github prepares and runs specs and rubocop

With travis-ci transitioning to a limited credit system after Dec 31st and currently taking hours for jobs to go from queued to completion,  I think we'll need to migrate all our CI to github actions
I use a custom docker-compose which includes this anyways.  mysql is also necessary for github actions so making this the default.
@pgwillia pgwillia changed the title [WIP] Use github workflows to run CI Use github workflows to run CI Dec 22, 2020
@pgwillia pgwillia requested review from jefferya and a user December 22, 2020 23:19
RAILS_ENV: test
run: |
bundle exec rake db:create
bundle exec rake db:migrate
Copy link
Contributor

@murny murny Dec 23, 2020

Choose a reason for hiding this comment

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

Small tip, I think rails db:schema:load is a bit more performant than doing migrations (Will use the db/schema.rb to provision DB instead of running all the migrations). But that's totally up to you 👍

Copy link
Member Author

Choose a reason for hiding this comment

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

💯 You've definitely warned me about this before and I had hoped to fix it. Right now there is some seeding that tests depend on that is set up by migrations. It's a pain to deal with and I've learned my lesson.
#1932

Copy link
Contributor

@murny murny left a comment

Choose a reason for hiding this comment

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

Nice work 👍

@pgwillia pgwillia merged commit 63bbdc2 into master Dec 23, 2020
@pgwillia pgwillia deleted the github_actions branch December 23, 2020 21:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants