Autre #13294
MandayeJS : respecter PEP 8
100%
Fichiers
Révisions associées
Historique
Mis à jour par Josué Kouka il y a plus de 7 ans
- Fichier 0001-fix-pep8-errors-13294.patch 0001-fix-pep8-errors-13294.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 7 ans
y'a pas que du pep8, aussi pas mal de nettoyage d'imports ; sont-ils tous vérifiés ?
Mis à jour par Frédéric Péters il y a plus de 7 ans
if not set(('SITE_FORCE_REDIRECT_URL', 'SITE_FORCE_REDIRECT_LOCATOR'))\
Ce n'est pas abordé dans la PEP 8 mais je n'aime pas les \ collés.
Mis à jour par Josué Kouka il y a plus de 7 ans
Thomas Noël a écrit :
y'a pas que du pep8, aussi pas mal de nettoyage d'imports ; sont-ils tous vérifiés ?
Oui oui. Flake8 + Tests
Mis à jour par Josué Kouka il y a plus de 7 ans
Mis à jour par Thomas Noël il y a plus de 7 ans
Frédéric Péters a écrit :
if not set(('SITE_FORCE_REDIRECT_URL', 'SITE_FORCE_REDIRECT_LOCATOR'))\Ce n'est pas abordé dans la PEP 8 mais je n'aime pas les \ collés.
Allez je fais mon "python<2 fanboy", mais moi c'est surtout le « if not set(('SITE_FORCE_REDIRECT_URL', 'SITE_FORCE_REDIRECT_LOCATOR')).intersection(set(dct.keys())): » que je trouve peu lisible (par rapport à « if 'SITE_FORCE_REDIRECT_URL' is not in dct and 'SITE_FORCE_REDIRECT_LOCATOR' is not in dct: »)
Mis à jour par Thomas Noël il y a plus de 7 ans
Thomas Noël a écrit :
Allez je fais mon "python<2 fanboy", mais moi c'est surtout le « if not set(('SITE_FORCE_REDIRECT_URL', 'SITE_FORCE_REDIRECT_LOCATOR')).intersection(set(dct.keys())): » que je trouve peu lisible (par rapport à « if 'SITE_FORCE_REDIRECT_URL' is not in dct and 'SITE_FORCE_REDIRECT_LOCATOR' is not in dct: »)
MAIS ce n'est pas l'objet de ce patch, je sais.
Mis à jour par Josué Kouka il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
- % réalisé changé de 0 à 100
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
- Statut changé de Résolu (à déployer) à Fermé
fix pep8 errors and remove unused imports (#13294)