-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
[vcpkg] translate the README to french #13971
Conversation
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.
Here are some changes : the most of them are typo fixing, other are wording changes to better stick with the english readme, etc.
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.
it's done just some typos to fix
fix typos Co-authored-by: Antoine James Tournepiche <26577763+AntoineJT@users.noreply.github.com>
some new typo fix Co-authored-by: Antoine James Tournepiche <26577763+AntoineJT@users.noreply.github.com>
Co-authored-by: Jack·Boos·Yu <47264268+JackBoosY@users.noreply.github.com>
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.
Just a small nit (comma to period), feel free to disregard since I don't know French.
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.
A few nitpicks on things that look weird to me as a french person (maybe not to someone from Québec?), but otherwise looks good
Je m'appelle Bill. Merci! ... something something French. Also thanks to Olivier Giroux who double checked: https://twitter.com/__simt__/status/1322026346256560128 |
Describe the pull request
I translated the README to french to help people