Tech : remplacement du validateur SIRET custom par la gem siret_validator

[kemenaran]

Coucou les copaines ! 👋

Après avoir recopié pour la centième fois le code de siret_validator.rb dans un centième projet, je me suis enfin décidé en faire une gem siret_validator. Et maintenant je fais le tour des projets Rails qui valident des SIRET, pour voir si ça les intéresse :)

Avantages de la gem :

• Toute légère (un seul fichier de code),
• Bien testée,
• Du code en moins dans le projet,
• Meilleure validation des SIRET de la Poste (il s'avère qu'ils ont une forme de checksum, juste pas la même que les autres),
• i18n incluse,
• Évite d'avoir le même code copié-collé entre plein de projets (avec de subtiles variations à chaque fois),
• Chouette référence pour les projets futurs qui s'inspirent de DS.

Inconvénients d'utiliser la gem :

• Une dépendance de plus,
• Le code de base n'est pas bien lourd, ni pénible à maintenir dans le projet,
• Moins facile de trifouiller le code pour gérer des cas particuliers à DS (mais en même temps il ne devrait pas en y avoir).

Ça vous dit ?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.97%. Comparing base (8049498) to head (ca9268a).

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #11888      +/-   ##
==========================================
- Coverage   89.97%   89.97%   -0.01%     
==========================================
  Files        1303     1302       -1     
  Lines       28675    28657      -18     
  Branches     5938     5932       -6     
==========================================
- Hits        25801    25783      -18     
  Misses       2874     2874              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.