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

Update rubocop v0.52.0 - Remove SpaceInsideBrackets #20

Merged
merged 1 commit into from
Dec 19, 2017
Merged

Update rubocop v0.52.0 - Remove SpaceInsideBrackets #20

merged 1 commit into from
Dec 19, 2017

Conversation

joshRpowell
Copy link
Contributor

@joshRpowell joshRpowell commented Dec 15, 2017

Update with SpaceInsideArrayLiteralBrackets & SpaceInsideReferenceBrackets
per PR in v0.52.0 https://github.com/bbatsov/rubocop/pull/5149/files
@joshRpowell joshRpowell changed the title Update rubocop v0.52.0 Update rubocop v0.52.0 - Remove SpaceInsideBrackets Dec 15, 2017
@gjtorikian
Copy link
Contributor

Can you also update this PR so that the enforce style is set to no_space?

@gjtorikian
Copy link
Contributor

You know, I just realized #19 was opened to address this. Thanks fo your contribution, I'll try to get that PR moving along.

@gjtorikian gjtorikian closed this Dec 19, 2017
@joshRpowell
Copy link
Contributor Author

joshRpowell commented Dec 19, 2017 via email

@gjtorikian gjtorikian reopened this Dec 19, 2017
@gjtorikian gjtorikian merged commit 87b32d7 into github:master Dec 19, 2017
@joshRpowell
Copy link
Contributor Author

@gjtorikian still see SpaceInsideBrackets

Layout/SpaceInsideBrackets:
after merging #20. Perhaps there was a conflict after #19 was merged that wasn't resolved prior to merging #20. Or are we leaving in for versions < 0.51.0?

@gjtorikian
Copy link
Contributor

Argh, no, I just haven't had enough coffee. Thanks!

tijmenb pushed a commit to alphagov/govuk-lint that referenced this pull request Apr 4, 2018
@joshRpowell joshRpowell deleted the joshRpowell-v0.52.0 branch March 7, 2019 14:00
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