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

Simplify GitHub Action RuboCop by using globally installed gems #38834

Merged
merged 1 commit into from
Mar 27, 2020

Conversation

olleolleolle
Copy link
Contributor

@olleolleolle olleolleolle commented Mar 27, 2020

Summary

Install and run rubocop as a global gem to speed up and simplify the linting workflow.

Other Information

Attempt Time
Before 1min 54s
With this PR 1min 19s

@olleolleolle olleolleolle changed the title Simplify GitHub Action RuboCop Simplify GitHub Action RuboCop by using globally installed gems Mar 27, 2020
@rafaelfranca rafaelfranca merged commit 86eac9b into rails:master Mar 27, 2020
@olleolleolle olleolleolle deleted the simpler-workflow-rubocop branch March 27, 2020 20:59
@eugeneius
Copy link
Member

We're now installing the latest version of RuboCop rather than the one in Gemfile.lock. This means that when a new RuboCop version is published, the check may start to fail if any rules that we have enabled have changed their behaviour. Currently those changes can be dealt with as part of a PR bumping the version in Gemfile.lock (e.g. #37821, #36426).

What if we add a rubocop group to the Gemfile and use bundle install --only rubocop instead? Would that give us most of the speed benefit, without sacrificing reproducibility?

@kaspth
Copy link
Contributor

kaspth commented Mar 27, 2020

@eugeneius good point, worth a shot 🙌

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.

None yet

4 participants