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

Upgrade to Ruby 2.6.6 #987

Open
wants to merge 4 commits into
base: master
from

Conversation

@artfuldodger
Copy link

@artfuldodger artfuldodger commented Oct 16, 2020

Specs start failing with 2.7.1.

@CLAassistant
Copy link

@CLAassistant CLAassistant commented Oct 16, 2020

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@filipesperandio filipesperandio left a comment

I think this move should account for a couple other things too.
For ref: ac66e5a

Specs start failing with 2.7.1.
@artfuldodger artfuldodger force-pushed the artfuldodger:upgrade-to-ruby-2.6.6 branch from c915bbd to 09f7cad Oct 16, 2020
@artfuldodger
Copy link
Author

@artfuldodger artfuldodger commented Oct 16, 2020

Ah yup, thanks for the reference commit @filipesperandio! PR is updated.

Copy link
Collaborator

@filipesperandio filipesperandio left a comment

Thanks!
Mind checking the violations too?
https://codeclimate.com/github/codeclimate/codeclimate/pull/987

I also wouldn't mind if we update rubocop to a more recent version. (rubocop-0-92)
https://github.com/codeclimate/codeclimate/blob/master/.codeclimate.yml#L10

@artfuldodger artfuldodger force-pushed the artfuldodger:upgrade-to-ruby-2.6.6 branch from d2485d9 to 7fa25be Oct 16, 2020
@artfuldodger
Copy link
Author

@artfuldodger artfuldodger commented Oct 16, 2020

Fixed the violations, upgraded Rubocop, and addressed the new violations as well. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.