Lambda

[sunny]

Débat entre :

A.

scope :owned_by, ->(user) { where(owner_id:  user.id) }

B.

scope :owned_by, -> (user) { where(owner_id:  user.id) }

C.

scope :owned_by, -> user { where(owner_id:  user.id) }

[sunny]

C.

Parce que c'est plus succint. Je ne trouve pas que les parenthèses soient très utiles ici.

Par contre ça va à l'encontre des règles du Ruby Style Guide de base.

[AnneSottise]

C.

Parce que je pense comme Sunny (◕‿◕)

[Ynote]

C.

[cyrill62]

A.

Car ce sont les règles du style-guide de base.

Sinon j'aime plus le :

D. lambda do |user|

[bsartek]

C. J'ai pas de personnalité

[sunny]

@cyrill62 Tu préfèrerais remplacer tous les -> par des lambda à chaque fois ? o_O

[cyrill62]

Ouais je sais c'est chelou mais même si c'est plus verbeux je trouve ça plus lisible.

Comme les blocs.

[camilledel]

C. Je reste fidèle à mon équipe !

[ghost]

autre question :

-> (user) do
  ...
end

ou

-> (user) {
   ...
}

Nos rubocops se contredisent la dessus.

[cyrill62]

Ben :

lambda do |user|
...
enfin

ou

->(user) { ... }

[sunny]

Bien vu @TristeFigure. Tu peux faire une issue ?

[robertanisko]

Victoire du C !

[cyrill62]

Ok du coup on peut mettre à jour le rubocop.yml en conséquence ?

[ghost]

Surtout il faudrait avoir un rubocop.yml unique pour tous les repos.