Remove other_args
declaration if it is no used
#578
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When
inspect_args
function is disbaled (e.g. whenenv = production
), and there is no use forother_args
(catch_all
is not used), then shellcheck triggers a warning on this line:This PR:
Command#catch_all_used_anywhere?
method that checks if the command itself or any of its descendants is usingcatch_all
other_args
declaration.