-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add github action #17
Conversation
ThHareau
commented
Sep 16, 2020
- Add CI github action
Create workflow
Add only rubocop
* Using local actions * Add RuboCop * Limit to push master and pr * Pull request only * No dependency on test * Using real ruby versions * Update ruby version * options for PG? * Start service * Split * Change service * with env * Remove host * Stop starting the service * Trying other config * idea * Adding password to env * real test script * Add cache * Add rubocop * Install in dedicated script * Fix * Fix fix * Fix fix fix * remove needs
5d8ec23
to
3dd8ecf
Compare
- uses: doctolib/setup-ruby@v1 | ||
with: | ||
ruby-version: ${{ env.RUBY_VERSION }} | ||
- uses: doctolib/cache@v2 | ||
with: | ||
path: vendor/bundle | ||
key: ${{ runner.os }}-gems-${{ hashFiles('**/Gemfile.lock') }} | ||
restore-keys: | | ||
${{ runner.os }}-gems- | ||
- name: bundle | ||
run: | | ||
gem install bundler | ||
bundle config path vendor/bundle | ||
bundle install --jobs 4 --retry 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tried to move this part here in an independent action, but as document in actions/runner#438 it is not yet possible to use other actions in composite actions.
Apparently github is working on it, once done we could have an internal action in a repo that we can use in any of our repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would also make the file a lot shorter :)