J'ai laissé 21 coms en reviewant 290 lignes Python dans la PR de ma N+1
Le 22 juin 2022 à 20:37:53 :
Le 22 juin 2022 à 20:34:35 ChantN8 a écrit :
Le 22 juin 2022 à 20:33:18 :
Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression
Oui je suis plutôt doué en Python, c'est un peu ma deuxième langue natale.
Tu peux t'essayer à l'asyncio si tu maîtrises pas.
J'ai fait de l'async en Javascript et Rust, ça devrait aller je pense
Le 22 juin 2022 à 20:36:49 RoiCatholique a écrit :
Le 22 juin 2022 à 20:29:34 :
Le 22 juin 2022 à 20:26:43 RoiCatholique a écrit :
T'as fait ça sans qu'elle te l'ait demandé ?C'est quoi PR c'est un programme pour coder ?
Quand on code en équipe, avant de valider le changement (= Pull Request = PR), chaque membre de l'équipe est invité à regarder le code qui a été écrit pour déceler des éventuels bugs ou imprécisions.
Chacun laisse (ou non) des commentaires expliquant ce qui ne va pas, et après le codeur est chargé de mettre à jour son code ou répondre aux commentaires laissés.Imagine t'écrit ton code qui marche plutôt bien, tu le publies, et t'as un connard qui toutes les 10 lignes laisse un commentaire pour dire que t'as pas codé de manière optimale...
Si t'as vraiment raison sur l'optimisation du code t'as bien fait.
Même si t'es casse couille puisque tu te hisse au niveau de ta hiérarchie et que tu rajoute du travail, l'objectif de ce que vous produisez c'est que ça marche au mieux.
Ça peut être mal pris en mode mêle toi de ton cul ou à l'inverse tu peux être considéré comme un élément de grande valeur.
Je te comprends au fond moi sur un texte je ne pourrai pas m'empêcher de corriger les fautes et rendre plus élégantes les tournures de phrases foireuses.
Mais ce que je dis marche uniquement si t'es un bon gros matrixé par le code qui a un certain talent. Tu verras bien ce qu'elle a à en dire et si finalement c'est pas ta vision qui est mal optimisée pour des raisons que tu n'avais pas en tête.
Merci kheyou de ton avis objectif.
Oui le code je vois ça comme de la prose, donc je pense qu'on partage un peu la même vision.
Je sais pas vraiment si ça va m'être bénéfique, mais bon, on verra bien comme tu dis...
Le 22 juin 2022 à 20:37:42 AlbertoJhonson a écrit :
Comment vous faites les PR ?
Souvent j'en ai rien a faire le regarde vite fait et je merge
Il faut un minimum de 2 personnes qui approuvent la PR avant qu'elle soit merge.
Théoriquement, un reviewer peut aussi convertir ses commentaires en "tâches à compléter" et mettre un tag "require change before merge" mais je l'ai pas fait, faut pas pousser...
Le 22 juin 2022 à 20:40:02 ChantN8 a écrit :
Le 22 juin 2022 à 20:37:53 :
Le 22 juin 2022 à 20:34:35 ChantN8 a écrit :
Le 22 juin 2022 à 20:33:18 :
Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression
Oui je suis plutôt doué en Python, c'est un peu ma deuxième langue natale.
Tu peux t'essayer à l'asyncio si tu maîtrises pas.
J'ai fait de l'async en Javascript et Rust, ça devrait aller je pense
Les concepts très avancés c'est genre les meta-class en Python ou l'inhéritance multiple. Mais ça sert une fois tous les 5 ans.
Le 22 juin 2022 à 20:42:06 Kogba a écrit :
Si c'est vraiment juste un script utilitaire et que t'as seulement commenté des moyens d'optimiser le code, alors oui tu es casse-couilles
Bah c'est un script censé être utilisé au quotidien quand même quoi...
Mais oui je suis casse-couilles je sais bien. J'ai pas pu m'empêcher...
Le 22 juin 2022 à 20:42:06 :
Le 22 juin 2022 à 20:40:02 ChantN8 a écrit :
Le 22 juin 2022 à 20:37:53 :
Le 22 juin 2022 à 20:34:35 ChantN8 a écrit :
Le 22 juin 2022 à 20:33:18 :
Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression
Oui je suis plutôt doué en Python, c'est un peu ma deuxième langue natale.
Tu peux t'essayer à l'asyncio si tu maîtrises pas.
J'ai fait de l'async en Javascript et Rust, ça devrait aller je pense
Les concepts très avancés c'est genre les meta-class en Python ou l'inhéritance multiple. Mais ça sert une fois tous les 5 ans.
Ouais au final c'est comme dans les autres langages OOP
Merci khey
Le 22 juin 2022 à 20:44:06 ChantN8 a écrit :
Le 22 juin 2022 à 20:42:06 :
Le 22 juin 2022 à 20:40:02 ChantN8 a écrit :
Le 22 juin 2022 à 20:37:53 :
Le 22 juin 2022 à 20:34:35 ChantN8 a écrit :
Le 22 juin 2022 à 20:33:18 :
Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression
Oui je suis plutôt doué en Python, c'est un peu ma deuxième langue natale.
Tu peux t'essayer à l'asyncio si tu maîtrises pas.
J'ai fait de l'async en Javascript et Rust, ça devrait aller je pense
Les concepts très avancés c'est genre les meta-class en Python ou l'inhéritance multiple. Mais ça sert une fois tous les 5 ans.
Ouais au final c'est comme dans les autres langages OOP
Merci khey
Regarde aussi comment s'interfacer avec l'api C de Python c'est pratique ça en vrai.
Le 22 juin 2022 à 20:44:50 AutisteCouillu a écrit :
Le 22 juin 2022 à 20:44:06 ChantN8 a écrit :
Le 22 juin 2022 à 20:42:06 :
Le 22 juin 2022 à 20:40:02 ChantN8 a écrit :
Le 22 juin 2022 à 20:37:53 :
Le 22 juin 2022 à 20:34:35 ChantN8 a écrit :
Le 22 juin 2022 à 20:33:18 :
Le 22 juin 2022 à 20:31:48 ChantN8 a écrit :
Tu as bien fait khey, ça lui apprendra a coder de la merde
C'est Python en back ? Django/Flask ?C'est pas tant de la merde, c'est juste que c'est du code Python de débutant avec quelques erreurs et imprécisions.
C'est sûr que le code sera beau et propre mais mes relations professionnelles vont en prendre un coup...
C'est pas du backend c'est un script utilitaire.Ok je vois, tes calé en Python ? J'ai un entretien bientot niveau avancé tu sais sur quoi je peux m'entrainer, des concepts bien avancés ? Python ya rien très compliqué j'ai l'impression
Oui je suis plutôt doué en Python, c'est un peu ma deuxième langue natale.
Tu peux t'essayer à l'asyncio si tu maîtrises pas.
J'ai fait de l'async en Javascript et Rust, ça devrait aller je pense
Les concepts très avancés c'est genre les meta-class en Python ou l'inhéritance multiple. Mais ça sert une fois tous les 5 ans.
Ouais au final c'est comme dans les autres langages OOP
Merci kheyRegarde aussi comment s'interfacer avec l'api C de Python c'est pratique ça en vrai.
Avoues-le que tu voudrais t'interfacer avec elle
L'OP t'as jamais eu envie de faire ton propre OS, ton propre langage de code, ton propre navigateur, etc, et même ton propre hardware ?
Genre en passion à long terme comme un mecano qui renove son Alpine 1965 toute rouillée pendant 15 ans le week-end
Ou l'informatique évolue trop vite pour créer ton propre environnement seul ?
Le 22 juin 2022 à 20:44:23 uients a écrit :
Tu as des conseils un peu generaux a donner sur comment faire de bons scripts utilitaires ?
Bah, suivre les principes SOLID déjà de base, et je rajouter de limiter au maximum la mutabilité des variables, privilégier les fonctions pure, bien séparer chaque étape, etc.
Pour tes scripts, fait une bonne description avec argparse, gère tous les cas d'erreurs (n'ignore pas les fonctions qui peuvent échouer), et surtout utilise du logging à foison et des messages d'erreur explicite (avec du contexte) quand il y a un problème.
Utilise le type hint et de la documentation.
C’est pas plutôt des merge request que vous faites ?
Le pull c’est juste quand tu récupères le code.
Le 22 juin 2022 à 20:47:07 RoiCatholique a écrit :
L'OP t'as jamais eu envie de faire ton propre OS, ton propre langage de code, ton propre navigateur, etc, et même ton propre hardware ?Genre en passion à long terme comme un mecano qui renove son Alpine 1965 toute rouillée pendant 15 ans le week-end
Ou l'informatique évolue trop vite pour créer ton propre environnement seul ?
Propre langage de code j'avais déjà fait à l'époque.
Propre navigateur / OS là par contre c'est trop bas niveau pour que j'y prenne du plaisir, ça demande des années et des années de travail.
Propre hardware oui j'aimerais bien un jour. Mais juste le design d'un circuit imprimé sur lequel je pourrais souder mes composant (genre Rasberry Pi custom). Je voulais faire une console portable d'emulation comme ça.
Mais bon, quand tu deviens ingesclave tu finis par manquer de temps...
Le 22 juin 2022 à 20:52:04 Dextre355 a écrit :
C’est pas plutôt des merge request que vous faites ?Le pull c’est juste quand tu récupères le code.
Oui, c'est un terme qui porte à confusion chez les débutants.
Chez Gitlab ils utilisent le terme "merge request" mais sur Github/Bitbucket ils utilisent le terme "pull request".
Je suis d'accord que c'est très mal nommé, mais c'est bien comme çe que ça s'appelle. Ça représente la même chose.
Données du topic
- Auteur
- AutisteCouillu
- Date de création
- 22 juin 2022 à 20:22:30
- Nb. messages archivés
- 76
- Nb. messages JVC
- 76