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

Actualize rubocop version and fix issues #86

Closed
wants to merge 23 commits into from
Closed

Actualize rubocop version and fix issues #86

wants to merge 23 commits into from

Conversation

ShockwaveNN
Copy link
Contributor

I've tried to fix as many issues as able, but feel free to mention code styles changed with maintainer team is not agree, I'm able to revert almost all of this changes without much troubles

@ShockwaveNN
Copy link
Contributor Author

ShockwaveNN commented Nov 20, 2020

UPD2: All those non actual since parallel released v1.20.1

Ok, CI seems fine except for ruby 2.4 - can't handle parallel gem required by rubocop, but I think I can get a workaround to use older one

Ruby 2.4 is EOLed a half-year ago

I can try to fix it's required version or just removed it

UPD: Fixed, looks pretty safe for me 6490ebc

@ShockwaveNN
Copy link
Contributor Author

@pointlessone could you please take a look, I'm gladly receive any kind of feedback )

@ShockwaveNN
Copy link
Contributor Author

Any news?

@pointlessone
Copy link
Member

pointlessone commented Dec 29, 2020

Thank you for your contribution.

I have implemented a shared Rubocop config (among other things) and had to test it myself a bunch so I ended up not using your PR.

@ShockwaveNN ShockwaveNN deleted the feature/fix-rubocop branch December 30, 2020 07:28
@ShockwaveNN
Copy link
Contributor Author

@pointlessone Ok, fair enough, thanks for new release with ruby 3.0 support

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

Successfully merging this pull request may close these issues.

None yet

2 participants