Fix Rubocop violations

[spencertransier]

What does it do?

• This PR fixes some of rubocop violations from .rubocop_todo.yml. I went through and removed a section from the todo file, ran Rubocop, then fixed any of the issues it found.
• It may be helpful to review this PR commit by commit to see which rubocop rule affected which changes.
• I also regenerated .rubocop_todo.yml to get the latest status by running bundle exec rubocop --regenerate-todo
• For the Naming/VariableName changes, I verified that those variables are only used internally by those actions and are not breaking changes for clients.

Checklist before requesting a review

• Run bundle exec rubocop to test for code style violations and recommendations
• Add Unit Tests (aka specs/*_spec.rb) if applicable
• Run bundle exec rspec to run the whole test suite and ensure all your tests pass
• Make sure you added an entry in the CHANGELOG.md file to describe your changes under the appropriate existing ### subsection of the existing ## Trunk section.
• If applicable, add an entry in the MIGRATION.md file to describe how the changes will affect the migration from the previous major version and what the clients will need to change and consider.

[crazytonyli]

I trust Rubocop made the right auto-correct changes. And the name changes looks good to me too.

[iangmaia]

Nice one, @spencertransier 👍

LGTM. I'll wait for this one to land before I finally merge #489 to avoid conflicts on your side, so that I can also update the code according to the updated rules over there.

[spencertransier]

Thanks @iangmaia. Sounds good, I'm excited to have your PR merged!