Closed adriweb closed 2 years ago
Si c'est ok pour toi, je peux résoudre les conflits et merger la PR.
J'ai corrigé quelques petites erreurs, (comme je l'adapte en C# ça me permet de trouver quelques erreurs qui se cache par-ci par là).
C'est OK pour moi.
Mais dis moi tu as vraiment lu tout le code ? Parce que l'adaptation que tu as fait jamais j'aurais su qu'il était possible de faire ça. Avec les static etc...
Le jeu. 13 oct. 2022 à 00:57, Adrien Bertrand @.***> a écrit :
Si c'est ok pour toi, je peux résoudre les conflits et merger la PR.
— Reply to this email directly, view it on GitHub https://github.com/Hamza-STI/CASYMBA/pull/1#issuecomment-1276776830, or unsubscribe https://github.com/notifications/unsubscribe-auth/APJMQLNTB2SHSSW3L6WOKXDWC4X5XANCNFSM6AAAAAARBIPN7Y . You are receiving this because your review was requested.Message ID: @.***>
Non, pas lu tout le code (un jour peut être ^^) mais juste assez pour faire cette PR, c'est surtout juste histoire de faire comprendre (aux devs et au compilateur) ce qui est public et ce qui est privé (static) vu que ça aide à l'optimisation.
Pour mieux voir les changements, il faut cacher les whitespaces