No spaces after block's left curly or before block's right curly

[chriseppstein]

https://github.com/nex3/sass/blob/master/test/rubocop_extensions.rb

We prefer 10.times {puts 'hi'} So, basically the way we think about it is if there's a space in the block declaration it's attached to the right side of the argument list, not to the curly.

[bbatsov]

@jonas054 Would you handle this? I think we should probably have config params like RequireSpaceAfterOpeningParen and RequireSpaceBeforeClosingParen and set them to true by default.

[jonas054]

Yes!

[jonas054]

I just have a few things to get clear before I start. First of all, is it really necessary to be able to configure RuboCop to allow 10.times {puts 'hi' } and 10.times { puts 'hi'}? Or can we have just one parameter that allows leaving out the spaces inside (both) the braces?

[bbatsov]

Good point! We need to support just { something } and {something} as with hashes. Plus the special case that when { something } is the used style we have to also consider the variable controlling {|x| something } whether this style is acceptable or not.

[jonas054]

Thanks for clearing that up. My next question is whether not requiring space is enough. I think that if we allow, by configuration, to leave the spaces out ({something}), we should consider { something } an offence.

[bbatsov]

The check should be strict. If you said you want to enforce the style {something} then anything that's not it is an offence - { something }, { something} and {something }.

The EnforcedStyle params should have the options space inside braces and no space inside braces.