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

check_whitespace/arrow_alignment doesn't handle all cases properly #174

Open
gerases opened this issue Oct 30, 2023 · 1 comment
Open

check_whitespace/arrow_alignment doesn't handle all cases properly #174

gerases opened this issue Oct 30, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@gerases
Copy link

gerases commented Oct 30, 2023

Describe the Bug

Given code similar to below:

someresource { 'title':
  foo          => 'bar',
}

No warnings are raised

Expected Behavior

I would expect the extra white space to be flagged, but I realize this might have been outside the scope of this plugin. If this is the case, can you recommend the plugin needed?

@gerases gerases added the bug Something isn't working label Oct 30, 2023
@gerases
Copy link
Author

gerases commented Nov 1, 2023

Upon further contemplation, I realized that it's probably outside the scope of this arrow_alignment plugin. So I wrote a separate plugin to handle this case. I will submit a PR within the coming days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant