Alignement des blocs inline

[sunny]

Débat entre :

A.

# Customer
let(:customer) { create :customer, :mangopay }
let(:legal_representative_country_of_residence) { 'DE' }

# User
let(:email) { 'mangopay-customer-test@example.com' }
let(:visible) { true }

B.

# Customer
let(:customer)                                  { create :customer, :mangopay }
let(:legal_representative_country_of_residence) { 'DE' }

# User
let(:email)                                     { 'mangopay-customer-test@example.com' }
let(:visible)                                   { true }

C.

# Customer
let(:customer)                                  { create :customer, :mangopay }
let(:legal_representative_country_of_residence) { 'DE' }

# User
let(:email)   { 'mangopay-customer-test@example.com' }
let(:visible) { true }

D.

Autoriser un mélange A et C.

[sunny]

A.

Parce que :

• B et C je ne vois ça que chez nous et je ne vois ça adopté par aucun autre projet Ruby (ou JS, CSS, …).
• B et C génèrent des lignes très longues qui arrivent bien plus vite au 80 chars
• Ça évite un espace trop important entre la clef et sa définition
• Ça évite de devoir choisir à entre B et C
• Ça évite de devoir décaler l'ensemble des règles à chaque fois qu'on ajoute une ligne plus longue que le reste
• Ça rends la revue de code plus lisible (même débat sur les diff que #1)

[cyrill62]

B.

Vous me faites mal aux yeux avec le A et le C.

[sunny]

Suite au commentaire de @bsartek j'ai mis à jour les exemples pour bien faire comprendre la différence de "paragraphes" entre B et C.

[cyrill62]

C.

Pardon. Le nouvel exemple est plus clair.

[robertanisko]

C.

[bsartek]

C.

[Ynote]

A.

[AnneSottise]

C.

[karineHe]

C.

[cyrill62]

Du coup je rejoint la team A pour ces mêmes raisons : https://github.com/KissKissBankBank/ruby-style-guide/issues/4#issuecomment-339996995

Ce qui veut dire aussi que je rejoint la team :

a = 1
ab = 2
abcd = 3
b = 5

[LanF3usT]

A.

[begault]

A