Problème avec Github - flake8

Version imprimable

Bonjour

Malgré que mon code fonctionne sans erreur avec moi
Quand je fais un commit vers mon Github, il trouve des choses qui ne vont pas.

Voici la dernière phrase des commentaires :
Code:

1 2 3 4 5 6 7 ./venv/Lib/site-packages/pkg_resources/_vendor/pyparsing.py:169:19: F821 nom non défini 'unicode' 152 ret = unicode(obj).encode(sys.getdefaultencoding(), 'xmlcharrefreplace') 153 ^ 154 48 F821 nom non défini 'unicode' 155 48 156 Erreur: Processus terminé avec le code de sortie 1.
Soit je répare moi-même "difficilement" mon code "qui fonctionne".
Ou, vous me proposez une solution pour éviter cette erreur.
Merci, car j'aimerais éviter ce genre de plantage.

13/08/2022, 10h32
papajoker
bonjour

Tout est dit dans ton erreur le message d'erreur unicode(obj), cette classe (ou fonction ?) n'existe pas dans python

facile à reproduire ;)
Code:

1 2 3 4 Traceback (most recent call last): File "/home/patrick/test.py", line 7, in <module> ret = toumic("2.0") NameError: name 'toumic' is not defined
EDIT:
Dans le code source de pyparsing, on peux voir que unicode n'est en fait qu'un alias

Code:

from .unicode import pyparsing_unicode as unicode

Je suppose que, avec ton commit, git n'utilise pas ton python local (ou l'inverse), donc ne trouve pas bien la bibliothèque ?
Je suppose que ton hook pre-commit ne tient pas compte de l'environnement virtuel de python
13/08/2022, 17h16
toumic2.0

Je te fais confiance au sujet de ton analyse, mais à mon niveau que puis-je faire ?
Ce qui n'est peut-être pas grave le fait que l'environnement virtuel ne soit pas dans le Github ?
15/08/2022, 11h14
papajoker

Citation:

Envoyé par toumic2.0

Je te fais confiance au sujet de ton analyse

Je n'ai fait que jouer avec ma boule de cristal et ... malheureusement, tu ne confirmes aucune de mes hypothèses (en cascades)

Citation:

peut-être pas grave le fait que l'environnement virtuel ne soit pas dans le Github ?

? désolé mais je ne comprends pas la question tu as un truc spécifique github et non git ? Git n'est pas github et github n'est pas Git ! Peut-être que c'est quelque chose que tu as pioché au hasard, mais si tu n'en parles pas, pas de risque de deviner.
Si ta config utilise un environnement virtuel, c'est voulu par toi et n'est qu'une configuration locale (donc par exemple pas dans "github actions").
Tout à coup, j'ai un doute ... l'environnement virtuel est bien voulu ?

Citation:

mais à mon niveau que puis-je faire ?

Si, par miracle, j'ai raison, c'est que tu as une configuration très avancée git/python. Ce qui semble complètement aberrant vu ton niveau qui semble plutôt débutant.
Il est facile de faire une recherche sur les hooks git et donc de voir si tu utilises cette technique avancée. Et pourquoi le fais-tu ? Si tu le désires, alors tu corriges, sinon tu les supprimes
15/08/2022, 12h11
toumic2.0

Citation:

Envoyé par papajoker

Je n'ai fait que jouer avec ma boule de cristal et ... malheureusement, tu ne confirmes aucune de mes hypothèses (en cascades)

? désolé mais je ne comprends pas la question tu as un truc spécifique github et non git ? Git n'est pas github et github n'est pas Git ! Peut-être que c'est quelque chose que tu as pioché au hasard, mais si tu n'en parles pas, pas de risque de deviner.
Si ta config utilise un environnement virtuel, c'est voulu par toi et n'est qu'une configuration locale (donc par exemple pas dans "github actions").
Tout à coup, j'ai un doute ... l'environnement virtuel est bien voulu ?

Si, par miracle, j'ai raison, c'est que tu as une configuration très avancée git/python. Ce qui semble complètement aberrant vu ton niveau qui semble plutôt débutant.
Il est facile de faire une recherche sur les hooks git et donc de voir si tu utilises cette technique avancée. Et pourquoi le fais-tu ? Si tu le désires, alors tu corriges, sinon tu les supprimes

Le pyparsing.py cause le problème F821 dans Actions · Toumic/GlobalApplis (github.com)
Je dois donc trouver la solution dans mon espace client et régler mes préférences.

J'ai donc désactivé les actions.
15/08/2022, 12h20
papajoker

ok, ma boule de cristal est cassée :mouarf:

Ce n'est pas un problème de hook git mais un problème github action
https://github.com/Toumic/GlobalAppl...python-app.yml sans requirements.txt, ne risque pas de fonctionner

ps: mettre le répertoire /venv/ dans le repo git n'a aucun sens, c'est justement pour cela qu'existe requirements.txt
16/08/2022, 08h02
toumic2.0

Citation:

Envoyé par papajoker

ok, ma boule de cristal est cassée :mouarf:

Ce n'est pas un problème de hook git mais un problème github action
https://github.com/Toumic/GlobalAppl...python-app.yml sans requirements.txt, ne risque pas de fonctionner

ps: mettre le répertoire /venv/ dans le repo git n'a aucun sens, c'est justement pour cela qu'existe requirements.txt

Je n'arrive pas à voir quelques chose dans les bouts de cristal de ta boule
J'avoue ne pas comprendre requirements.txt, au final je dois mettre requirements.txt dans le /venv/ sur mon ordi ?

J'ai visité : https://help.github.com/actions/language-and-framework-guides/using-python-with-github-actions

Étant un débutant confirmé et au vu de toutes les choses à lire et comprendre avant de pouvoir exercer mes pouvoirs sur mon python-app.yml.
Je crains de ne pas pouvoir corriger la version de Python qui est maintenant 3.10, alors que mon python-app.yml indique 3.9...
16/08/2022, 10h26
fred1599

Bonjour,

Si tout est fonctionnel, alors simplement rajouter # noqa: F821 à ta ligne de code qui pose problème et ça sera tout simplement ignoré.
16/08/2022, 11h27
papajoker

Il faudrait déjà comprendre pourquoi tu utilises cet outil, puisque tu n'as pas l'air de le comprendre ... ??? c'est "un truc" complètement optionnel et plutôt réservé au devs qui maitrisent.
C'est principalement pour effectuer les tests que nous avons écrits ... et il me semble que tu n'en as pas écrit un seul :roll:

venv : ce sont les paquets python que tu as installé en local ;)
requirements.txt : c'est la liste des paquets que tu as installé en local pour faire tourner ce programme ! tu dis justement à ton fichier de config github d'installer les paquets qui sont listés dans ce fichier ;)

Au lieu de fournir simplement ce fichier, tu fournis ton répertoire /venv/, ce qui est complètement inutile ; on ne le met jamais dans son dépôt git !
Puisque tu n'installes aucun paquet (requirements.txt n'existe pas), (si besoin de modules externes) il est donc impossible que les tests de ton programme puissent fonctionner dans ton conteneur github
16/08/2022, 11h57
toumic2.0

Citation:

Envoyé par papajoker

Il faudrait déjà comprendre pourquoi tu utilises cet outil, puisque tu n'as pas l'air de le comprendre ... ??? c'est "un truc" complètement optionnel et plutôt réservé au devs qui maitrisent.
C'est principalement pour effectuer les tests que nous avons écrits ... et il me semble que tu n'en as pas écrit un seul :roll:

venv : ce sont les paquets python que tu as installé en local ;)
requirements.txt : c'est la liste des paquets que tu as installé en local pour faire tourner ce programme ! tu dis justement à ton fichier de config github d'installer les paquets qui sont listés dans ce fichier ;)

Au lieu de fournir simplement ce fichier, tu fournis ton répertoire /venv/, ce qui est complètement inutile ; on ne le met jamais dans son dépôt git !
Puisque tu n'installes aucun paquet (requirements.txt n'existe pas), (si besoin de modules externes) il est donc impossible que les tests de ton programme puissent fonctionner dans ton conteneur github

_{C'est principalement pour effectuer les tests que nous avons écrits ... et il me semble que tu n'en as pas écrit un seul}
Vu comme ça, je vais supprimer cette option qui traite de mes tests (absents)

_{requirements.txt : c'est la liste des paquets que tu as installé en local pour faire tourner ce programme ! tu dis justement à ton fichier de config github d'installer les paquets qui sont listés dans ce fichier}
Si j'ai écris des modules\__init__.py == paquets (cooriges-moi si je me trompe), c'est pour rendre exécutable ce programme (si je veux).
- modules\__init__.py = paquets ?

_{Au lieu de fournir simplement ce fichier, tu fournis ton répertoire /venv/, ce qui est complètement inutile ; on ne le met jamais dans son dépôt git !}
Ce que je fournis via le commit, ça se fait tout seul avec mon Pycharm. Je ne sais pas pourquoi on ne le fait jamais.

Citation:

Envoyé par fred1599

Bonjour,

Si tout est fonctionnel, alors simplement rajouter # noqa: F821 à ta ligne de code qui pose problème et ça sera tout simplement ignoré.

Bonjour

J'ai bien mis # noqa: F821 , au bout de la ligne ./venv/Lib/.../pyparsing.py:169:19:
Code:

1 2 3 4 5 6 7 ./venv/Lib/site-packages/pkg_resources/_vendor/pyparsing.py:169:19: F821 nom non défini 'unicode' 152 ret = unicode(obj).encode(sys.getdefaultencoding(), 'xmlcharrefreplace') 153 ^ 154 48 F821 nom non défini 'unicode' 155 48 156 Erreur: Processus terminé avec le code de sortie 1.
Mais seulement ce code ce n'est pas moi qui l'a écrit, et en plus il comporte un grand nombre d'erreurs à en croire mon Pycharm.

Je vais donc me coller à ajouter à chaque fois # noqa: F821
Je reviendrais sur lce sujet (noqa) quand j'en aurais terminé
.../...
Au final ça a l'air de marcher, mais j'arrête cette tentative de rectification
Mais aussi, qui me dit que ces rectifications ne gênent pas le cours normal des codes écrits par les autres ?

17/08/2022, 10h01
fred1599

Citation:

qui me dit que ces rectifications ne gênent pas le cours normal des codes écrits par les autres ?

Ça ne gêne en rien, mais je ne vois pas pourquoi flake s'attaque à contrôler tes librairies python dans ton dossier venv. Il doit y avoir un moyen d'exécuter flake sans prendre en compte ce dossier.
18/08/2022, 04h36
toumic2.0

1 pièce(s) jointe(s)

Citation:

Envoyé par fred1599

Ça ne gêne en rien, mais je ne vois pas pourquoi flake s'attaque à contrôler tes librairies python dans ton dossier venv. Il doit y avoir un moyen d'exécuter flake sans prendre en compte ce dossier.

J'abandonne les démarches sont clairement insuffisantes pour un novice

Pièce jointe 623957