Rubocop

[gregoirenovel]

No description provided.

[kemenaran]

Merci pour le coup de ménage sur le linter.

Je suis assez partant pour les règles proposées (vous en pensez quoi, vous autres ?), à l'exception de :

• Les trailing-commas : j'ai l'impression qu'il y a des projets où c'est au contraire une pratique recommandée. J'ai du mal à avoir un avis là dessus ; mais dans l'immédiat je me demande s'il faut vraiment partir complètement dans un sens ou dans l'autre. Vous avez un avis ?

• Enable Style/SymbolProc : j'aime bien la syntaxe concise – mais dans certains cas je trouve que la syntaxe étendue est plus claire. Par exemple pour le dernier exemple du commit (after(:create, &:renew_api_token)).

  Du coup je suis hésitant à forcer cette syntaxe un peu magique par défaut. Un avis ?

[tchak]

J'ai des doutes sur Enable Style/WordArray et Enable Style/SymbolArray. Ça vous dérange tent que ça ?

[tchak]

@kemenaran pour les trailing-commas, c'est effectivement un sujet très subjectif. Perso je déteste les trailing-commas. L'argument c'est que ça rend les diffs plus lisibles, mais perso j'ai l'impression de passer plus de temps à lire le code que les diffs.

[gregoirenovel]

Pour les *Array :

• j'ai toujours du mal à me souvenir de la signification des raccourcis %w, %i, etc.
• je trouve que c'est un proxy cognitif dont on peut se passer
• je trouve que ça pête parfois la recherche car j'ai tendance à mettre les quotes lorsque je recherche (pour éviter de matcher des noms de variables par ex)

[gregoirenovel]

@kemenaran d'accord avec toi pour after(:create, &:renew_api_token) j'ai ajusté la conf pour ne pas avoir à écrire cela