Ajouter le super-linter à la CI

[HAEKADI]

• Amélioration technique
• Zones impactées : configuration de l'intégration continue.
• Détails :
  • Ajouter le super-linter à la configuration de la CI

[bonjourmauko]

Hello ! Je vois que cela change un peu les styles, en particulier les hang indents et le Knuth's style pour les opérations multi-ligne, est-ce un consensus de la communauté ? 😧

[HAEKADI]

Hello @maukoquiroga !
Pas à ma connaissance. Donc j'ai arrêté les changements, d'où ma question sur le consensus (sur Country Template), parceque OpenFisca France n'utilise pas pylint non plus. Donc je n'ai pas les idées claires quant aux normes choisies 🤔

[bonjourmauko]

Je pense que les règles présentes sur country-template et extension-template sont une bonne assisse, et elles peuvent être intégrées au superlinter de ce que j'ai pu lire vite fait.

[HAEKADI]

Ok très bien, @maukoquiroga tu parles bien des règles définies ici ?

@sandcha @MattiSG @benjello Vous avez plus de connaissance de l'historique peut-être. Pensez-vous que c'est le moment de clarifier les règles de linting ? 🤔
@maukoquiroga a déjà traité le sujet sur Openfisca-Core.

[benjello]

@HAEKADI oui on peut tout à fait les expliciter. Les choix retenus par @maukoquiroga reflètent bien l'histoire des débats (on a bien discuté de cela quand il a été question d'utiliser un formateur de code).

[HAEKADI]

J'ai utilisé la config country-template dans le linting de la CI.

[MattiSG]

Suite à la décision prise dans openfisca/openfisca-core#1040, on va préférer uniformiser les usages de la CI pour s'appuyer sur la CLI et un gestionnaire de tâches plutôt que d'avoir des commandes spécifiques à la CI, ce que serait superlinter.

Par ailleurs, on voit que l'évolution de la configuration du linting est un sujet à part entière, qui devrait être porté indépendamment, probablement comme une RFC, ce qui alourdira significativement le processus.

En l'état, comme vu avec @HAEKADI ce matin par visio, je crois qu'il vaut mieux l'abandonner.

Cette PR souligne au passage un point intéressant : le badge CircleCI n'est pas à jour et aurait du être effacé dans #1663 😉