Skip to content

Disable Rubocop Performance/TimesMap

username-removed-423915 requested 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