Cop idea: enforce a style for include, extend, prepend, etc.

[backus]

I see two distinct styles for mixing in modules:

Style 1

class Foo
  extend Bar
  extend Baz
  include Qux
  include Norf
  include Grault
  include Thud
end

Style 2

class Foo
  extend Bar, Baz
  include Qux, Norf, Grault, Thud
end

It would be nice to be able to enforce either of these styles

[bbatsov]

I agree. That's a good idea. It would also need a style guide rule.

Same applies to the attr_* family of methods, btw.

[backus]

Thanks for implementing @Drenmi!

[PikachuEXE]

Note that the grouped style reverses the ancestor order
Not sure if worth mentioning

include TestModule1
include TestModule2
#=> TestModule1 before TestModule2

include TestModule1, TestModule2
include TestModule2
#=> TestModule2 before TestModule1

# You can run this in irb:
module TestModule1; end; module TestModule2; end; 
class TestClass1; include TestModule1, TestModule2; end; TestClass1.ancestors
class TestClass2; include TestModule1; include TestModule2; end; TestClass2.ancestors

[Drenmi]

@PikachuEXE: Wow. Great catch! My assumptions were totally wrong here. 😅 Patch incoming.