Phildem / Cath

Simple cooperative multitask helper for Arduino
GNU General Public License v3.0
8 stars 3 forks source link

Fixes english comments & indentation #1

Closed soif closed 3 years ago

soif commented 3 years ago

Bonjour Philippe.

Pour te remercier de ton fantastique travail pédagogique, voici un petit PR rapide qui corrige certaines typo en anglais. C'est sans doute pas encore parfait, mais ca devrait un peu améliorer la lecture.

J'ai egalement fixé quelques indentations.

En esprérant que ça te soit utile,

Cordialement

Phildem commented 3 years ago

Hello,

Merci pour ces corrections.

soif commented 3 years ago

you're welcome!

BTW ca serait super de transformer ca en lib : Enfin une lib simple, legere, pour faire du multiTask!

soif commented 3 years ago

Re-Salut Philippe

Je vois que tu as "mergé" mon PR (sans doute depuis GH) dans le commit c415438311bb56d1107803e5c9cdcde8ba8479e6

Puis juste après tu as "pullé" le commit 2ed714a412c19ae7eb829657c9dc20e74682b05f qui a exactement annulé le commit précédent...

Comme tu avait l'ait d'avoir accepté le commit précédent, c'est sans doute une erreur de manip de ta part :-) Je devine que tu as accepté le PR sur GH, (ce qui l'as mergé sur ta branche "REMOTE"), mais que tu ne l'as pas mergé en local dans TA branche Master, et que tu as refait un commit, de la version précédente.... ;-)

Si c'est volontaire, no offence, :-), you're the boss !!! sinon, il faut soit "reverse" ton dernier commit, soit si tu préfere je refait le meme PR...

Phildem commented 3 years ago

Hello,

Non, c’est une erreur… Je ne suis pas un as de Git 😊

Mais ça ne compilais plus alors j’ai voulu faire une correction.

Merci de bien vouloir :

Ajouter dans le release History :

1.4 13/09/2021 Fixes english comments & indentation

Et surtout bien vérifier que l’appli compile toujours.

Le plus simple est en effet de refaire un pull request.

Merci et bonne journée

@.***

Philippe Demerliac

Visitez mon site http://www.cyrob.org http://www.cyrob.org

Et regardez ma chaine YouTube : https://www.youtube.com/c/PhilippeDemerliac_Cyrob https://www.youtube.com/c/PhilippeDemerliac_Cyrob

Demandez à rejoindre le groupe Facebook : Cyrob.org

Le groupe discord Cyrob : https://discord.gg/mNMSE53 https://discord.gg/mNMSE53

Soutenir Cyrob.org : http://philippe.demerliac.free.fr/Support.htm http://philippe.demerliac.free.fr/Support.htm

De : soif @.> Envoyé : dimanche 12 septembre 2021 21:42 À : Phildem/Cath @.> Cc : Philippe Demerliac @.>; State change @.> Objet : Re: [Phildem/Cath] Fixes english comments & indentation (#1)

Re-Salut Philippe

Je vois que tu as "mergé" mon PR (sans doute depuis GH) dans le commit https://github.com/Phildem/Cath/commit/c415438311bb56d1107803e5c9cdcde8ba8479e6 c415438

Puis juste après tu as "pullé" le commit https://github.com/Phildem/Cath/commit/2ed714a412c19ae7eb829657c9dc20e74682b05f 2ed714a qui a exactement annulé le commit précédent...

Comme tu avait l'ait d'avoir accepté le commit précédent, c'est sans doute une erreur de manip de ta part :-) Je devine que tu as accepté le PR sur GH, (ce qui l'as mergé sur ta branche "REMOTE"), mais que tu ne l'as pas mergé en local dans TA branche Master, et que tu as refait un commit, de la version précédente.... ;-)

Si c'est volontaire, no offence, :-), you're the boss !!! sinon, il faut soit "reverse" ton dernier commit, soit si tu préfere je refait le meme PR...

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/Phildem/Cath/pull/1#issuecomment-917697984 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRB3LEQDUMGSGMF67UKJL3UBT7BTANCNFSM5D3472VQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

soif commented 3 years ago

Voir suite dans #2