-
-
Notifications
You must be signed in to change notification settings - Fork 263
Library csv #1091
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Library csv #1091
Conversation
awecx
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour ta première PR.
Attention aux espaces insécables qui peuvent casser le RST (il faut modifier padpo sur ce point).
Je t'ai mis quelques remarques à traiter avant de pouvoir fusionner.
| #: ../Doc/library/csv.rst:269 | ||
| msgid "An example for :class:`Sniffer` use::" | ||
| msgstr "Un exemple d'utilisation de :class:`Sniffer` : ::" | ||
| msgstr "Un exemple d'utilisation de :class:`Sniffer` : ::" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| msgstr "Un exemple d'utilisation de :class:`Sniffer` : ::" | |
| msgstr "Un exemple d'utilisation de :class:`Sniffer` :" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| msgstr "Un exemple d'utilisation de :class:`Sniffer` : ::" | |
| msgstr "Un exemple d'utilisation de :class:`Sniffer` ::" |
Attention là tu rajoutes une espace insécable entre « : » et « :: ».
Tu peux cependant remplacer par « SnifferNBSP:: ». Idem dans la suite du fichier.
| msgstr "" | ||
| "Écrit une ligne avec les noms de champ (comme spécifié dans le constructeur) " | ||
| "dans l'objet fichier du transcripteur, formatée selon *dialect*. Renvoie la " | ||
| "valeur de retour de l'appel :meth:`csvwriter.writerow` utilisé en interne." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| "valeur de retour de l'appel :meth:`csvwriter.writerow` utilisé en interne." | |
| "valeur de retour de l'appel à :meth:`csvwriter.writerow` utilisée en interne." |
| "writer's file object, formatted according to the current dialect. Return the " | ||
| "return value of the :meth:`csvwriter.writerow` call used internally." | ||
| msgstr "" | ||
| "Écrit une ligne avec les noms de champ (comme spécifié dans le constructeur) " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'aurais mis :
| "Écrit une ligne avec les noms de champ (comme spécifié dans le constructeur) " | |
| "Écrit une ligne avec le nom (passé au constructeur) de chaque champ " |
pour bien expliciter qu'on passe le nom des champs au constructeur, pas les champs eux-mêmes.
| ":meth:`writeheader` now also returns the value returned by the :meth:" | ||
| "`csvwriter.writerow` method it uses internally." | ||
| msgstr "" | ||
| ":meth:`writeheader` renvoie aussi maintenant la valeur retournée par la " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| ":meth:`writeheader` renvoie aussi maintenant la valeur retournée par la " | |
| ":meth:`writeheader` renvoie aussi maintenant la valeur renvoyée par la " |
| #: ../Doc/library/csv.rst:486 | ||
| msgid "The simplest example of reading a CSV file::" | ||
| msgstr "Le plus simple exemple de lecture d'un fichier CSV : ::" | ||
| msgstr "Le plus simple exemple de lecture d'un fichier CSV : ::" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
À modifier donc.
| msgstr "Le plus simple exemple de lecture d'un fichier CSV : ::" | |
| msgstr "Le plus simple exemple de lecture d'un fichier CSV ::" |
Padpo me semble globalement correct, il n'aime pas (enfin, il est censé 😉)
Par contre il aime
Le point qui n'est pas correct dans padpo est qu'il n'aime pas
@awecx Tu as d'autres exemple de faux positifs en tête ? |
|
Ping :-) |
@vpoulailleau J'avais raté cette question... |
|
@awecx Pas de souci. Pour info, cela a été mis dans le ticket vpoulailleau/padpo#17 (EDIT : en fait c'était déjà marqué, mais je viens juste de le voir 😛) |

Fix #1088
First pull request