flotpython / course

Contenu pédagogique du MOOC Python3 sur fun-mooc.fr
Other
130 stars 22 forks source link

suppression des espaces en fin de ligne #36

Closed aollier closed 4 years ago

aollier commented 4 years ago

fichiers .py

parmentelat commented 4 years ago

bonjour Adrien merci mais franchement c'est le genre de manipe qui fait un peu peur ! 80 fichiers modifiés... est-ce que c'est bien raisonnable ? est-ce que tu vas avoir le temps de vérifier que ça ne casse rien ?

aollier commented 4 years ago

Bonsoir Thierry,

J'ai vérifié qu'à l'intérieur d'une définition de classe, s'il y a une ligne avec ou sans espaces, l'interpréteur Python l'ignore. Et donc une ligne ne comportant qu'un retour à la ligne (au sein de la définition d'une classe) ne provoque pas d'erreur. La plupart des corrections c'est d'enlever des espaces après une suite de caractères, espaces qui sont donc totalement superflus et bien sûr ça ne casse rien. Une des modifications c'est aussi d'enlever les retours à la ligne superflus en fin de fichier. Ça ne peut rien casser non plus.

Bien sûr je n'ai pas testé les 80 fichiers mais ce type de correction est totalement inoffensif. Il n'y a pas de crainte à avoir.

aollier commented 4 years ago

Bonjour Thierry,

je n'ai pas d'environnement de test mais je te confirme que les modifications que j'ai apportées ne cassent rien. J'ai pu vérifier que les lignes vides sans espaces ne provoquent pas d'erreur au niveau de l'interpréteur Python (ce qui me paraît tout à fait souhaitable) et tout le reste c'est la suppression d'espaces en fin de ligne ou de retours à la ligne superflus.

Tu peux fusionner mes modifications sans crainte. Adrien