Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Correction de typos #1791

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

Merged
merged 2 commits into from
Dec 16, 2021
Merged

Correction de typos #1791

merged 2 commits into from
Dec 16, 2021

Conversation

flashcode
Copy link
Contributor

No description provided.

Copy link
Collaborator

@jeanas jeanas left a comment

Choose a reason for hiding this comment

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

Merci !

faq/gui.po Outdated
@@ -59,7 +59,7 @@ msgid ""
msgstr ""
"D'autres outils existent. Le choix doit dépendre notamment de la ou des "
"plateformes que vous visez. Sur le Wiki Python se trouvent des listes de "
"bibliothèques graphiques `multiplate-formes <https://wiki.python.org/moin/"
"bibliothèques graphiques `multi-plateformes <https://wiki.python.org/moin/"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Je me rappelle avoir choisi ce terme et il me semblait bien que cette forme était correcte. Cela dit, « multi-plateformes » n'a pas l'air faux non plus.

Copy link

Choose a reason for hiding this comment

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

« plates-formes » est le terme correct même si « plateformes » est toléré depuis la réforme de 1990.
Il me semble qu'il faille mettre « plates » au pluriel car c'est un adjectif.
On ne met pas de trait d'union après « multi » (sauf si le mot qui suit commence par i, ce qui n'est pas le cas ici).
Le terme correct est donc pour moi « multiplates-formes » ou « multiplateformes ».
J'invoque @christopheNan pour un avis.

Copy link
Contributor

Choose a reason for hiding this comment

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

Puisque l'on me demande mon avis : je trouve « multiplates-formes » un peu bizarre. Je préfère donc « multiplateformes ».

Copy link
Collaborator

Choose a reason for hiding this comment

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

« multiplateformes » me convient très bien aussi.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'ai donc corrigé pour utiliser le terme "multiplateformes" partout.
Et pour info j'ai systématiquement utilisé la forme plurielle, je ne suis pas certain que ça fasse sens parfois d'utiliser "multiplateforme" sans "s".

glossary.po Outdated
@@ -762,7 +762,7 @@ msgstr ""

#: glossary.rst:325
msgid "dictionary comprehension"
msgstr "dictionnaire en compréhension (ou dictionnaire en intension)"
msgstr "dictionnaire en compréhension (ou dictionnaire en intention)"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Non, pour le coup, c'était correct. Ce sens de « intension » s'oppose à « extension ».

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exact, autant pour moi, je vais corriger les deux occurences modifiées.

@jeanas
Copy link
Collaborator

jeanas commented Dec 14, 2021

@flashcode Merci d'avoir fait les corrections. Pour info, la pratique dans ce projet est plutôt d'ajouter des commits que de faire git commit --amend ou git rebase -i et git push --force ou git push --force-with-lease. C'est plus facile ainsi pour les relecteurs de visualiser les corrections. De toutes façons, toutes les PRs sont fusionnées en squash-merging.

jeanas
jeanas previously approved these changes Dec 14, 2021
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 cette PR, je suis d'accord avec tous les changements sauf le « multiplates-formes ».

faq/gui.po Outdated
@@ -59,7 +59,7 @@ msgid ""
msgstr ""
"D'autres outils existent. Le choix doit dépendre notamment de la ou des "
"plateformes que vous visez. Sur le Wiki Python se trouvent des listes de "
"bibliothèques graphiques `multiplate-formes <https://wiki.python.org/moin/"
"bibliothèques graphiques `multi-plateformes <https://wiki.python.org/moin/"
Copy link

Choose a reason for hiding this comment

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

« plates-formes » est le terme correct même si « plateformes » est toléré depuis la réforme de 1990.
Il me semble qu'il faille mettre « plates » au pluriel car c'est un adjectif.
On ne met pas de trait d'union après « multi » (sauf si le mot qui suit commence par i, ce qui n'est pas le cas ici).
Le terme correct est donc pour moi « multiplates-formes » ou « multiplateformes ».
J'invoque @christopheNan pour un avis.

@flashcode
Copy link
Contributor Author

C'est noté @jean-abou-samra, je suis d'accord, je vais éviter les push force.
Merci pour la relecture des modifs.

@awecx awecx merged commit 6f13a7a into python:3.10 Dec 16, 2021
@awecx
Copy link

awecx commented Dec 16, 2021

Merci @flashcode 🥧

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.

4 participants