Skip to content

Library csv#1091

Closed
hboueix wants to merge 3 commits into
python:3.8from
hboueix:library-csv
Closed

Library csv#1091
hboueix wants to merge 3 commits into
python:3.8from
hboueix:library-csv

Conversation

@hboueix

@hboueix hboueix commented Dec 13, 2019

Copy link
Copy Markdown

Fix #1088
First pull request

@Seluj78 Seluj78 added the NEED REVIEWER This PR needs a reviewer :) label Dec 13, 2019
@awecx awecx removed the NEED REVIEWER This PR needs a reviewer :) label Dec 13, 2019

@awecx awecx left a comment

Copy link
Copy Markdown

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.

Comment thread library/csv.po
#: ../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` : ::"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
msgstr "Un exemple d'utilisation de :class:`Sniffer` : ::"
msgstr "Un exemple d'utilisation de :class:`Sniffer` :"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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 « :: ».

Ça ne fonctionnera pas :
image

Tu peux cependant remplacer par « SnifferNBSP:: ». Idem dans la suite du fichier.

Comment thread library/csv.po
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."

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"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."

Comment thread library/csv.po
"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) "

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

J'aurais mis :

Suggested change
"É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.

Comment thread library/csv.po
":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 "

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
":meth:`writeheader` renvoie aussi maintenant la valeur retournée par la "
":meth:`writeheader` renvoie aussi maintenant la valeur renvoyée par la "

Comment thread library/csv.po
#: ../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 : ::"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

À modifier donc.

Suggested change
msgstr "Le plus simple exemple de lecture d'un fichier CSV : ::"
msgstr "Le plus simple exemple de lecture d'un fichier CSV ::"

@vpoulailleau

vpoulailleau commented Dec 15, 2019

Copy link
Copy Markdown
Contributor

Attention aux espaces insécables qui peuvent casser le RST (il faut modifier padpo sur ce point)

Padpo me semble globalement correct, il n'aime pas (enfin, il est censé 😉)

  • <space>:<space>::
  • <NBSP>:<NBSP>::
  • <space>::

Par contre il aime

  • <NBSP>::

Le point qui n'est pas correct dans padpo est qu'il n'aime pas

  • <NBSP>:<space>:: => vpoulailleau/padpo#17

@awecx Tu as d'autres exemple de faux positifs en tête ?

@awecx

awecx commented Dec 17, 2019

Copy link
Copy Markdown

Ping :-)

@awecx

awecx commented Dec 18, 2019

Copy link
Copy Markdown

@awecx Tu as d'autres exemple de faux positifs en tête ?

@vpoulailleau J'avais raté cette question...
Pas vraiment à part ceux discutés dans #1082 (comment).

@vpoulailleau

vpoulailleau commented Dec 19, 2019

Copy link
Copy Markdown
Contributor

@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 😛)

@hboueix hboueix closed this Jan 3, 2020
@hboueix hboueix deleted the library-csv branch January 3, 2020 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Je travaille sur library/csv.po

4 participants