search

ens-lyon-2017 / compass2018

c'est du sabotage/de l'exploitation
0 stars 1 forks source link

Bugfixes #19

Closed moy closed 6 years ago

moy commented 6 years ago

Bonjour,

Je suis le collègue de Laure sur le cours de CAP. Voici quelques bugfixes dont nous aurions besoin pour le cours.

L'historique n'est pas 100% clean (c'est une extraction de notre dépôt privé où on a tatonné un peu), mais le diff global est raisonablement trivial.

Merci d'avance,

lephe commented 6 years ago

Super, merci ! Pas mal d'étourderies qui méritaient d'être corrigées.

L'histoire du print("\n") automatique était pour tester un des programmes de @lauregonnord, et j'ai oublié de l'enlever...

Le fichier myasm.py semble effectivement être une vieille version de l'assembleur, je ne sais toujours pas pourquoi Alain ne l'a pas supprimé. J'ai arrêté de le mettre à jour quand j'ai fini par comprendre, ce qui m'a pris un bon moment.

moy commented 6 years ago

Pas mal d'étourderies qui méritaient d'être corrigées.

Ça va, la densité d'étourderies est bien plus élevée dans le code d'étudiants que j'ai l'habitude de lire ;-).

L'histoire du print("\n") automatique était pour tester un des programmes de @lauregonnord, et j'ai oublié de l'enlever...

En fait dans l'historique de ma pull-request il est ajouté puis supprimé.

Le fichier myasm.py semble effectivement être une vieille version de l'assembleur, je ne sais toujours pas pourquoi Alain ne l'a pas supprimé.

Je suis resté conservateur en ajoutant un warning, mais on pourrait le supprimer carrément. Je suis tombé dessus en faisant un "grep" pour comprendre ces histoires de sgt/gt/ge/... et j'ai perdu un peu de temps à comprendre que je n'étais pas au bon endroit.

lephe commented 6 years ago

En fait dans l'historique de ma pull-request il est ajouté puis supprimé.

Donc finalement je ne l'ai pas oublié, ouf !

Je suis resté conservateur en ajoutant un warning, mais on pourrait le supprimer carrément. Je suis tombé dessus en faisant un "grep" pour comprendre ces histoires de sgt/gt/ge/... et j'ai perdu un peu de temps à comprendre que je n'étais pas au bon endroit.

Je suis assez chaud pour le supprimer (j'ai hésité pour être honnête), j'aurais juste aimé avoir une confirmation verbale d'Alain avant. Enfin l'historique Git étant là, c'est peut-être mieux de s'en défaire tout de suite.