Skip to content

Conversation

@hboueix
Copy link

@hboueix hboueix commented Dec 13, 2019

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
Copy link

@awecx awecx left a 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` : ::"
Copy link

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

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.

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

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

"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

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.

":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

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 "

#: ../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

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
Copy link
Contributor

vpoulailleau commented Dec 15, 2019

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
Copy link

awecx commented Dec 17, 2019

Ping :-)

@awecx
Copy link

awecx commented Dec 18, 2019

@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
Copy link
Contributor

vpoulailleau commented Dec 19, 2019

@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