Projet

Général

Profil

Bug #74090

interval.py: remplacer le "raise Exception" par un bon "raise QuelqueChose"

Ajouté par Thomas Noël il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Catégorie:
-
Version cible:
-
Début:
02 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Pour passer les vérifications du nouveau pylint 2.16 d'hier, j'ai ajouté un noqa :

--- a/chrono/interval.py
+++ b/chrono/interval.py
@@ -196,7 +196,7 @@ class IntervalSet:
                         yield (c1[0], c2[0])
                     state = 3
                 else:
-                    raise Exception('not reachable')
+                    raise Exception('not reachable')  # noqa pylint: disable=broad-exception-raised

         return self.__class__.from_ordered(gen())

Idéalement il faudrait remplacer Exception par ValueError ou KeyError ou... en fait je ne savais pas très bien par quoi (d'où mon ajout du noqa en mode poussière-sous-le-tapis).

Si quelqu'un·e comprend ce bout de code mieux que moi et voit quelle exception envoyer, ça sera plus joli.

Révisions associées

Révision c78147d2 (diff)
Ajouté par Lauréline Guérin il y a environ un an

misc: raise AssertionError instead of Exception (interval) (#74090)

Historique

#1

Mis à jour par Frédéric Péters il y a environ un an

AssertionError.

#2

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Lauréline Guérin

Lauréline Guérin (lguerin) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Robot Gitea il y a environ un an

  • Assigné à changé de Lauréline Guérin à Thomas Noël
#4

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution proposée à Solution validée

Thomas NOEL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Thomas Noël il y a environ un an

  • Assigné à changé de Thomas Noël à Lauréline Guérin
#7

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)

Lauréline Guérin (lguerin) a mergé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF