[go: up one dir, main page]

Disable Rubocop Performance/TimesMap

Lin Jen-Shinrequested to merge
disable-rubocop-times-map into master

Disable Rubocop Performance/TimesMap

This cop would complain:

C: Performance/TimesMap: Use Array.new with a block instead of .times.map.

I don't think this makes sense. What if we really want to have sequential numbers? We could do: 0.upto(9).map but how that's so bad than using 10.times.map? Performance doesn't really matter here, we're certainly not running this in a loop. It's generally used for generating some testing data only.

@rymai @rspeicher @grzesiek What do you think?

Merge request reports