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
Remove some unused method parameters #21722
Remove some unused method parameters #21722
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just left a comment regarding a seemingly outdated comment, and potentially unnecessary changes to the Gemfile.lock
😊
I always like cleaning up dead code 😁
7a61ea2
to
ad4bcbc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
ad4bcbc
to
2fd836e
Compare
* Reformat code for readability * Remove unused env parameter in cruise_lane calls * Remove unused parameter * Remove unused parameter * Remove unused parameter
I made a first pass at running rubocop on unused method parameters and found these. Also a little code indentation cleanup.
All changes are in individual commits.
More improvements on the same topic coming later.
Checklist
bundle exec rspec
from the root directory to see all new and existing tests passbundle exec rubocop -a
to ensure the code style is validci/circleci
builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)Motivation and Context
Description
Testing Steps