search

dataforgoodfr / 12_bloom

23 stars 11 forks source link

Lots of tables #134

Closed njouanin closed 5 months ago

njouanin commented 5 months ago

Ajout des dernières tables #131

rv2931 commented 5 months ago

Hello Pas eu le temps de regarder mais tu sais pourquoi ça ne compilenplus en python 3.9/3.10 ?

rv2931 commented 5 months ago

Veirfie si ce n'est pas un problème de syntaxe de typing

njouanin commented 5 months ago

@rv2931 oui on a échangé avec @hehlinge42 sur slack. il doit ouvrir un ticket (#132) et corriger sur sa branche.

rv2931 commented 5 months ago

Je vois que tu as fait pas mal de tâches dans src/bloom/tasks Je te propose d'intégrer le mécanisme présenté dans la PR: https://github.com/dataforgoodfr/12_bloom/pull/124 Non pas que ça révolutionne la chose mais le fait de le gérer avec des classes permettra d'ajouter plus facilement le logging et la gestion d'erreurs Je peux m'en charger ce soir si ok