Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Move extra_args to the end of the rubocop command #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Move extra_args to the end of the rubocop command #19

wants to merge 1 commit into from

Commits on Oct 27, 2016

  1. Move extra_args to the end of the rubocop command 

    For some rubocop executables, a `--file` or `--path` option might be
necessary to target a specific file to run the parser.  This currently
would need to be the last argument in the string to be paired up with
the `l:filename` when running the RuboCop command.

Swapping these arguments should retain the previous functionality just
fine, while allowing for the use-case described above.

Note:  It was chosen to have `l:extra_args` last in the list versus
`a:current_args` since a `--file` option is more useful to be applied to
every command instead of on demand via the `a:current_args`.
    @NickLaMuro
    NickLaMuro committed Oct 27, 2016
    Configuration menu
    Copy the full SHA
    5c56174 View commit details
    Browse the repository at this point in the history