Fixes rubocop violation Lint/AmbiguousOperator

[aidamanna]

Summary

Fixes rubocop violation Lint/AmbiguousOperator.

Details

Method arguments have been parenthesised when using block operators or splat operators.

Motivation and Context

This change is required to satisfy the ruboxop requirement related to Lint/AmbiguousOperator.
Refs #1021

How Has This Been Tested?

No new tests. Existing tests are green (with bundle exec rake). Rubocop is happy.

Screenshots (if appropriate):

N/A.

Types of changes

• Refactor (code change that does not change external functionality)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Rubocop fix.

Checklist:

• I've added tests for my code
• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[olleolleolle]

👍 Thanks for a careful change.

[mxygem]

Thank you for the work here @aidamanna! I'm sorry to say that you and I aren't the only ones who've done this work but it's unneeded. :( This fix was initially done in #1022 but it was deemed that we wouldn't make these changes. Give the issue a read for more info!

Again, thank you for the work! If you'd like to grab another cop to work on please do. I need to get #1208 merged so that we don't run into this issue again. Sorry!

[mxygem]

@aidamanna If you pull down or look at the current code, the rubocop.yml and rubocop_todo.yml files are up to date now!

[mxygem]

@aidamanna I've updated master to be up to date now. Go give it a look and pull another style cop to fix? Sorry again that you ended up duplicating work!

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.