Alignement des conditions

[cyrill62]

A :

def test?
  true ||
  true
end

def test2?
  true &&
  true
end

def test3?
  true &&
  true ||
  true
end

B :

def test?
  true ||
    true
end

def test2?
  true &&
    true
end

def test3?
  true &&
    true ||
      true
end

C :

def test?
  true ||
  true
end

def test2?
  true &&
    true
end

def test3?
  true &&
    true ||
    true
end

D :

Comme rubocop le demande par défaut.

def test?
  true ||
    true
end

def test2?
  true &&
    true
end

def test3?
  true &&
    true ||
    true
end

Pour info, la discussion qui a mis le feu aux poudres 💥 😄

[robertanisko]

A, parce que je trouve pas ça logique d'indenter des branches qui sont au même niveau d'une expression.

[cyrill62]

D : pcq j'aime pas toucher à la config de rubocop 😃

Sinon A.

[cyrill62]

J'ai pas réussi à configurer rubocop pour qu'il accepte la A...

[sunny]

Comme @cyrill62, D pour suivre la norme, sinon A.

[bsartek]

D, l'aspect monolithique de A. me dérange visuellement