Skip to content

Update RuboCop version#178

Merged
kapoorlakshya merged 1 commit intotitusfortner:masterfrom
utkarsh2102:update-rubocop
Aug 11, 2020
Merged

Update RuboCop version#178
kapoorlakshya merged 1 commit intotitusfortner:masterfrom
utkarsh2102:update-rubocop

Conversation

@utkarsh2102
Copy link
Contributor

Hi @kapoorlakshya,

Thanks for taking care of this so far! 🚀
However, there was one offense left, which is what this PR fixes.
(and also updates to use latest RuboCop!

Signed-off-by: Utkarsh Gupta <utkarsh@debian.org>

@kapoorlakshya
Copy link
Collaborator

Hi @utkarsh2102, thank you for your contribution!

I just fixed the Metrics/AbcSize violation in IEdriver#downloads (see d43c29f). Can you please revert the file exclusion here?

I also fixed the unit test failure blocking your PR (see 6d39862), so please rebase on top of master and push. I'd be happy to approve and merge after. Thanks again!

@utkarsh2102
Copy link
Contributor Author

Oh wow, your fix for the violation is pretty neat! 💯

Signed-off-by: Utkarsh Gupta <utkarsh@debian.org>
@utkarsh2102 utkarsh2102 changed the title Update RuboCop and make Metrics/AbcSize happy Update RuboCop version Aug 10, 2020
@kapoorlakshya
Copy link
Collaborator

@utkarsh2102 Thank you for updating the PR! Merging...

@kapoorlakshya kapoorlakshya merged commit 2cf2725 into titusfortner:master Aug 11, 2020
@utkarsh2102 utkarsh2102 deleted the update-rubocop branch August 12, 2020 06:49
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.

2 participants