Vérifier que le fichier cron.json est correctement trié

[rsebille]

🤔 Pourquoi ?

Car c'était PAS TRIÉ ! 😁

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
pypi/python-crontab@3.2.0	environment, eval, filesystem, shell	0	233 kB	doctormo

🚮 Removed packages: pypi/regex@2023.12.25)

View full report↗︎

[celine-m-s]

[celine-m-s]

Marie Kondo a de la concurrence.

[francoisfreitag]

Je trouve que ça va un peu loin, ajouter une dépendance (même de test) me semble un peu overkill pour trier un fichier (même si la clé n’est effectivement pas évidente). 🤷

[rsebille]

Je trouve que ça va un peu loin, ajouter une dépendance (même de test) me semble un peu overkill pour trier un fichier (même si la clé n’est effectivement pas évidente). 🤷

C'est parce que le master plan c'est d'aussi utiliser cette dépendance pour lier cron.json et le @monitor() de Sentry 😁.

Mais oui c'est un peu overkill en l'état 😅, j'étais plutôt sur la piste de ne pas trier par fréquence car c'était relou mais vu que la lib le faisais nativement je me suis dit pourquoi pas, je trouve ça plus utile que juste l'ordre d’exécution dans la journée car ça explicite le comportement (et les problèmes) plutôt que de devoir y penser quand on fait ou modifie la management command.

J'ai aussi vu une méthode .is_valid() dans la lib donc ça pourra aussi servir à vérifier qu'on écrit pas trop nawak dedans, même si on a pas trop eu de problème à cause de ça pour le moment.

[xavfernandez]

Je trouve que ça va un peu loin, ajouter une dépendance (même de test) me semble un peu overkill pour trier un fichier (même si la clé n’est effectivement pas évidente). 🤷

Assez d'accord. Je voulais proposer dans un premier temps de faire le tri sur le nom de la commande (et donc sans la dépendance) mais c'est vrai que le tri par fréquence est plus lisible/logique.

Après pour aller dans le sens de l'ajout de la dépendance, on peut se dire qu'elle valide également la syntaxe du cron (et du coup je suis plutôt 0+ :) )

[rsebille]

Si personne ne s'insurge alors je fusionnerais demain :).

[francoisfreitag]

Le master plan me plaît bien, alors je suis prêt à payer la dépendance :)