Development #32866
nettoyage PEP8, codestyle, python3
0%
Description
Pré-nettoyage avant intégration future dans authentic.
Fichiers
Demandes liées
Révisions associées
improve code style (#32866)
- remove obviously dead code (reported by flake8)
- fix PEP8 violations
- rename variable using stdlib builtin names
- use get_version() from combo's setup.py
py3ize obviously non-compatible code (#32866)
add copyright headers (#32866)
improve code style (#32866)
- remove obviously dead code (reported by flake8)
- fix PEP8 violations
- rename variable using stdlib builtin names
- use get_version() from combo's setup.py
py3ize obviously non-compatible code (#32866)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0002-improve-code-style-32866.patch 0002-improve-code-style-32866.patch ajouté
- Fichier 0003-py3ize-obviously-non-compatible-code-32866.patch 0003-py3ize-obviously-non-compatible-code-32866.patch ajouté
- Fichier 0001-add-copyright-headers-32866.patch 0001-add-copyright-headers-32866.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
C'est le printemps, on nettoye.
Mis à jour par Emmanuel Cazenave il y a presque 5 ans
Histoire de chipoter ou d'apprendre quelque chose, pourquoi :
# duck-type unicode/Py3 strings
if hasattr(key, 'isdecimal'):
key = key.encode('utf-8')
plutôt que
# duck-type unicode/Py3 strings
if hasattr(key, 'encode'):
key = key.encode('utf-8')
?Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Emmanuel Cazenave a écrit :
Histoire de chipoter ou d'apprendre quelque chose, pourquoi :
[...]
plutôt que
[...]
?
Parce que
Python 2.7.15+ (default, Feb 3 2019, 13:13:16) [GCC 8.2.0] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> 'encode' in (set(dir(str)) & set(dir(unicode))) True >>> set(dir(unicode)) - set(dir(str)) set(['isnumeric', 'isdecimal'])
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0002-improve-code-style-32866.patch 0002-improve-code-style-32866.patch ajouté
- Fichier 0003-py3ize-obviously-non-compatible-code-32866.patch 0003-py3ize-obviously-non-compatible-code-32866.patch ajouté
- Fichier 0001-add-copyright-headers-32866.patch 0001-add-copyright-headers-32866.patch ajouté
- Fichier 0004-int-gration-nettoyage-cot-a2-32866.patch 0004-int-gration-nettoyage-cot-a2-32866.patch ajouté
- Fichier 0005-tox.ini-utilisation-de-l-authentic-du-ticket-32934.patch 0005-tox.ini-utilisation-de-l-authentic-du-ticket-32934.patch ajouté
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Bloqué par Development #32934: grand ménage de printemps ajouté
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
commit beca608c97c0ac15d5a96d8d5a363bbb114d74f0
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu May 9 19:19:18 2019 +0200
adapts to spring cleaning in authentic (#32866)
See #32934.
commit 2746d095c5f7eb08d7ef587937b870756e59f0d0
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue May 7 14:31:07 2019 +0200
py3ize obviously non-compatible code (#32866)
commit 007721868528d61cad165753a409f0996ebae831
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue May 7 14:07:59 2019 +0200
improve code style (#32866)
- remove obviously dead code (reported by flake8)
- fix PEP8 violations
- rename variable using stdlib builtin names
- use get_version() from combo's setup.py
commit e04459825c1f317cd71afd82ba2640cbcc7576dc
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue May 7 14:05:39 2019 +0200
add copyright headers (#32866)Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
add copyright headers (#32866)