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.

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

[FEATURE REQUEST] Style/SingleSpaceBeforeFirstArg:AllowForAlignment #2504

Closed
aeberlin opened this issue Dec 15, 2015 · 5 comments
Closed

[FEATURE REQUEST] Style/SingleSpaceBeforeFirstArg:AllowForAlignment #2504

aeberlin opened this issue Dec 15, 2015 · 5 comments

Comments

@aeberlin
Copy link

There exists this configuration option for Style/ExtraSpacing:

Style/ExtraSpacing:
  AllowForAlignment: true

Can we add the same configurability for Style/SingleSpaceBeforeFirstArg ?

Useful example:

.user-form
  = simple_form_for User.new do |f|
    = f.inline_input   :full_name,     as: :string
    = f.disabled_input :password,      as: :password
    = f.masked_input   :zip_code,      as: :string
    = f.masked_input   :email_address, as: :email
    = f.masked_input   :phone_number,  as: :tel
@alexdowad
Copy link
Contributor

Just pushed a fix for this one.

@aeberlin
Copy link
Author

@alexdowad Awesome! Thanks!

@aeberlin
Copy link
Author

@alexdowad I've been getting this warning, but I'm using the most recent release (0.36)

Warning: unrecognized cop Style/SingleSpaceBeforeFirstArg found in foo/.rubocop.yml

Thoughts? Thanks

@alexdowad
Copy link
Contributor

It's called SpaceBeforeFirstArg now. If my open PR is merged, RC will print a more helpful message. (See 6397b14.)

@aeberlin
Copy link
Author

Ahhh! Gotcha, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants