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

Style/SpaceInsideRangeLiteral should allow newlines #1406

Closed

CyborgMaster opened this issue Oct 23, 2014 · 0 comments

CyborgMaster commented Oct 23, 2014

Sometimes, when the parts of a range literal are long, it is best to split them up over multiple lines. While I agree that this is bad:

a ... b

I have the following in my code:

      @trigger.check([DateTime.parse('2014-05-12 04:00')...
                      DateTime.parse('2014-05-13 04:00')])

which RuboCop flags as incorrect. Is there a better way to do what I need, or do we need to improve the cop?

bbatsov closed this as completed in

85f2b8d

Bhacaz mentioned this issue

Add Extensions Cops Documentation references in the official docs. #7400

Merged

8 tasks

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