search

zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Met à jour Sentry et corrige un avertissement #6623

Closed philippemilink closed 4 months ago

philippemilink commented 4 months ago

Sur une erreur de publication rapportée par Sentry, il y avait les avertissements suivants :

warning

On utilise mal la fonction add_breadcumb(). Mais en fait les appels à add_breadcumb() (qui ajoutent des infos au fil d'Ariane) n'ajoutaient aucune information supplémentaire, puisque les appels à Popen sont déjà interceptés par Sentry, qui les ajoute directement au fil d'Ariane :

breadcumb

Donc, j'ai tout simplement enlevé les appels à add_breadcumb().

J'en ai profité aussi pour mettre à jour sentry-sdk. C'est un changement de version majeure, mais apparemment on n'a rien à changer.

Contrôle qualité

La CI fonctionne et peut-être que je testerai sur le serveur de bêta.

coveralls commented 4 months ago

Coverage Status

coverage: 88.842% (+0.01%) from 88.831% when pulling c6315bf8e998b5d9814da214cb487b278d0d87bb on philippemilink:remove-sentry-breadcumb-update into e3cf8e4f5f2dccdfb2da70c25e450d4916b935d8 on zestedesavoir:dev.