j'ai travaillé sur calendar #360

LiseG-tin · 2025-10-30T16:04:35+01:00

LiseG-tin commented

2025-10-30 16:04:35 +01:00

No description provided.

LiseG-tin added 1 commit

2025-10-30 16:04:36 +01:00

j'ai travaillé sur calendar

ci/woodpecker/pr/woodpecker Pipeline failed

Details

a22b899457

mdk requested changes

2025-10-30 22:16:49 +01:00

Dismissed

library/calendar.po Outdated

					
				@ -769,3 +769,3 @@

				"The :mod:`calendar` module can be executed as a script from the command line "

				"to interactively print a calendar."

				msgstr ""

				msgstr "Le module :mod:`calendar` peut être exécuté comme un script depuis la"

Il manque une espace à la fin ici, sinon la et ligne sont collés en laligne.

Il manque une espace à la fin ici, sinon `la` et `ligne` sont collés en `laligne`.

👍 1

LiseG-tin marked this conversation as resolved

library/calendar.po Outdated

					
				@ -785,3 +786,3 @@

				#: library/calendar.rst:647

				msgid "The locale to use for month and weekday names. Defaults to English."

				msgstr ""

				msgstr "Les paramètres régionaux à utiliser pour les noms de mois et de jour de semaine,"

Pareil, il manque une espace après ,.

Pareil, il manque une espace après `,`.

Et j'écrirais le nom des paramètres régionaux à utiliser …
https://docs.python.org/3/library/calendar.html#cmdoption-calendar-locale

Et j'écrirais *le nom des paramètres régionaux à utiliser* … https://docs.python.org/3/library/calendar.html#cmdoption-calendar-locale

LiseG-tin marked this conversation as resolved

library/calendar.po Outdated

					
				@ -791,3 +793,3 @@

				"The encoding to use for output. :option:`--encoding` is required "

				"if :option:`--locale` is set."

				msgstr ""

				msgstr "L'encodage à utiliser pour les données de sortie. :option:`--encoding` est"

pareil

LiseG-tin marked this conversation as resolved

library/calendar.po Outdated

					
				@ -795,3 +798,3 @@

				#: library/calendar.rst:659

				msgid "Print the calendar to the terminal as text, or as an HTML document."

				msgstr ""

				msgstr "Affiche le calendrier dans le terminal sous forme de texte ou de"

pareil

ChristopheNan marked this conversation as resolved

library/calendar.po Outdated

					
				@ -801,3 +805,3 @@

				"The weekday to start each week. Must be a number between 0 (Monday) and 6 "

				"(Sunday). Defaults to 0."

				msgstr ""

				msgstr "Le jour par lequel commence chaque semaine. Cela doit être un nombre entre 0 (lundi)"

pareil

ChristopheNan marked this conversation as resolved

ChristopheNan requested changes

2025-10-31 00:05:21 +01:00

library/calendar.po Outdated

					
				@ -773,3 +774,3 @@

				#: library/calendar.rst:592

				msgid "For example, to print a calendar for the year 2000:"

				msgstr ""

				msgstr "Par exemple, pour afficher un calendrier de l'année 2000 :"

une espace insécable est requise entre 2000 et les deux-points.
Voir https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/CONTRIBUTING.rst#caract%C3%A8res-sp%C3%A9ciaux-et-typographie pour insérer une espace insécable.

une espace **insécable** est requise entre *2000* et les deux-points. Voir https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/CONTRIBUTING.rst#caract%C3%A8res-sp%C3%A9ciaux-et-typographie pour insérer une espace insécable.

Je ne vois pas d'espace insécable.

LiseG-tin added 1 commit

2025-10-31 10:56:02 +01:00

correction des modifs sur calendar

ci/woodpecker/pr/woodpecker Pipeline failed

Details

5441660dd8

ChristopheNan commented

2025-10-31 11:29:03 +01:00

Merci pour la prise en compte des remarques.
Cependant, tu remarques que la validation automatique ne passe pas, en raison d'une longueur trop importantes de quelques lignes :

https://woodpecker.afpy.org/repos/29/pipeline/574/4#L194

Peux-tu remettre en forme le fichier pour que les lignes aient une longueur de moins de 80 caractères stp ?

Merci pour la prise en compte des remarques. Cependant, tu remarques que la validation automatique ne passe pas, en raison d'une longueur trop importantes de quelques lignes : https://woodpecker.afpy.org/repos/29/pipeline/574/4#L194 Peux-tu remettre en forme le fichier pour que les lignes aient une longueur de moins de 80 caractères stp ?

👍 1

ChristopheNan commented

2025-10-31 11:39:30 +01:00

Je ne connais pas ton niveau de maîtrise de la procédure de validation automatique. ATFU dans le cadre de ce sprint, je la décris ci-dessous :

La validation automatique consiste à dérouler les actions indiquées ici :

https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/.woodpecker.yml

Dans le cas présent, c'est lors du make check que l'erreur se produit. Le make check est décrit ici :

https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/Makefile#L208

C'est une bonne pratique de lancer sur son poste un make check avant de proposer une PR

Il consiste lui même à dérouler en dernière étape un make line-length qui est décrit ici :

https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/Makefile#L160

C'est-à-dire qu'il lance le script .scripts/line-length.py qui est ici :

https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/.scripts/line-length.py

Je ne connais pas ton niveau de maîtrise de la procédure de validation automatique. ATFU dans le cadre de ce sprint, je la décris ci-dessous : La validation automatique consiste à dérouler les actions indiquées ici : https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/.woodpecker.yml Dans le cas présent, c'est lors du *make check* que l'erreur se produit. Le *make check* est décrit ici : https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/Makefile#L208 **C'est une bonne pratique de lancer sur son poste un *make check* avant de proposer une PR** Il consiste lui même à dérouler en dernière étape un *make line-length* qui est décrit ici : https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/Makefile#L160 C'est-à-dire qu'il lance le script *.scripts/line-length.py* qui est ici : https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.14/.scripts/line-length.py

LiseG-tin added 1 commit

2025-10-31 12:09:18 +01:00

correction modif calendar

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

9dbec93b82

LiseG-tin commented

2025-10-31 12:19:30 +01:00

@ChristopheNan merci pour les liens, je vais regarder ça de ce pas

mdk approved these changes

2025-10-31 12:27:21 +01:00

mdk commented

2025-11-05 18:31:39 +01:00

@LiseG-tin rappel si tu en as besoin, pour mettre à jour ta branche :

git switch calendar  # Pour te remettre sur la bonne branche
poedit library/calendar.po  # Pour faire les modifs
git add library/calendar.po  # Pour préparer le commit
git commit -m "Proofreading by ChrisNan"
git push  # Pas besoin du -u, tu l'as déjà utilisé pour ouvrir la PR, maintenant git se souvient que `calendar` se push sur ton remote.

@LiseG-tin rappel si tu en as besoin, pour mettre à jour ta branche : git switch calendar # Pour te remettre sur la bonne branche poedit library/calendar.po # Pour faire les modifs git add library/calendar.po # Pour préparer le commit git commit -m "Proofreading by ChrisNan" git push # Pas besoin du -u, tu l'as déjà utilisé pour ouvrir la PR, maintenant git se souvient que `calendar` se push sur ton remote.

LiseG-tin commented

2025-11-05 18:33:56 +01:00

Merci ! Je ne pourrais peut-être m'en occuper que ce week-end ou la semaine prochaine malheureusement.

Lise Guillotin
Relectrice - correctrice
lise.guillotin@outlook.com
SIRET : 89177345900039
[cid:cc489c98-6fa2-452f-9776-955b449149e1]

Merci ! Je ne pourrais peut-être m'en occuper que ce week-end ou la semaine prochaine malheureusement. Lise Guillotin Relectrice - correctrice lise.guillotin@outlook.com SIRET : 89177345900039 [cid:cc489c98-6fa2-452f-9776-955b449149e1]

Outlook-ks2opixm.png

6.4 KiB