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

Don’t enforce %w[square brackets] for percent literals #9

Merged
merged 1 commit into from
Feb 16, 2018

Conversation

fanaugen
Copy link
Contributor

@fanaugen fanaugen commented Feb 14, 2018

As previously discussed here and there

Let’s treat each other like grown-ups: the difference between
%w(this style) and %w[that style] does not make or break readability,
but the cop does waste our time on every commit.

@zfhui
Copy link

zfhui commented Feb 14, 2018

I'm perfectly fine with this solution! ✅ Also see this issue.

@fanaugen
Copy link
Contributor Author

merging this now;

these commits are kept intentionally small ("atomic") for easy revert in case anyone should miss the cop in question and want it back

@fanaugen fanaugen merged commit ea02905 into master Feb 16, 2018
@fanaugen fanaugen deleted the copoff_percent_w branch February 16, 2018 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants