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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update rubocop from 1.12 to 1.18 and min ruby from 2.4 to 2.5 #8741

Merged
merged 1 commit into from
Jul 22, 2021

Conversation

andyjdavis
Copy link
Contributor

@andyjdavis andyjdavis commented Jul 21, 2021

This is a 馃敤 code refactoring.

Summary

This pull request does 3 related things:

  1. Update the version of rubocop from 1.12.0 to 1.18.3.
  2. Rubocop 1.18.3 doesnt support ruby 2.4 anymore. To allow rubocop to run this PR also brings the minimum ruby version up to 2.5.
  3. Autofix all of the problems reported by the newer version of rubocop.

Locally 'script/cibuild' gives this a clean bill of health.

Context

This will resolve multiple open issues.

Closes #8732
Closes #8726
Closes #8721
Closes #8720

@parkr parkr requested a review from DirtyF July 22, 2021 02:14
Copy link
Member

@DirtyF DirtyF left a comment

Choose a reason for hiding this comment

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

Thanks @andyjdavis

@DirtyF
Copy link
Member

DirtyF commented Jul 22, 2021

@jekyll: merge +minor

@jekyllbot jekyllbot merged commit 3f46f02 into jekyll:master Jul 22, 2021
jekyllbot added a commit that referenced this pull request Jul 22, 2021
github-actions bot pushed a commit that referenced this pull request Jul 22, 2021
Andrew Davis: Update rubocop from 1.12 to 1.18 and min ruby from 2.4 to 2.5 (#8741)

Merge pull request 8741
@jekyll jekyll locked and limited conversation to collaborators Jul 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants