Projet

Général

Profil

Development #56300

nouvelles erreurs pylint

Ajouté par Frédéric Péters il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
21 août 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Passage de pylint==2.9.6 à pylint==2.10.1 et nouveaux warnings https://jenkins.entrouvert.org/job/chrono/1483/pylint/new/

  • unspecified-encoding, ça demande maintenant aux open() de spécifier l'encodage, ça apparait à différents endroits perso je pense qu'on pourrait ignorer ces warnings, l'utf-8 par défaut étant ok.
  • Using variable 'e' before assignment sur la ligne : (je ne vois pas)
    events_by_slug = {e.slug: e for e in Event.objects.filter(agenda=self.agenda_pk)}
    
  • format-string-without-interpolation, sur err_class='unknown recurring event slug:' % event_slug,, manque un %s.

Fichiers

Révisions associées

Révision 694a51be (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

trivial: add missing interpolation to string (#56300)

Révision 993835e8 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

pylint: ignore unspecified-encoding which is specific to Windows (#56300)

Révision a6396507 (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

trivial: use x as list comprehension variable (#56300)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 2 ans

L'unspecified-encoding, ça vient de https://github.com/PyCQA/pylint/issues/3826, ça concerne juste windows et on s'en balance.

Le "Using variable 'e' before ..." c'est parce que plus loin dans la méthode on attribute une exception en l'appelant "e"; ça s'adapte en itérant plutôt sur une variable nommée x, totalement classique.

#2

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Assigné à mis à Frédéric Péters
#3

Mis à jour par Lauréline Guérin il y a plus de 2 ans

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

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit a63965070730f4e2f6b5247260a7522497bb8820
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Aug 21 09:55:47 2021 +0200

    trivial: use x as list comprehension variable (#56300)

commit 993835e8b6a9708dd55aac6a77ef49bb4ed1ba0e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Aug 21 09:54:18 2021 +0200

    pylint: ignore unspecified-encoding which is specific to Windows (#56300)

commit 694a51beef26d82619284e27bab91543ad9c2c01
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Aug 21 09:51:07 2021 +0200

    trivial: add missing interpolation to string (#56300)
#5

Mis à jour par Frédéric Péters il y a plus de 2 ans

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

Formats disponibles : Atom PDF