Skip to content
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

French language translation #1394

Merged
merged 4 commits into from
Jun 14, 2018
Merged

French language translation #1394

merged 4 commits into from
Jun 14, 2018

Conversation

PixiBixi
Copy link

Add french language to the vscode-python plugin :)

@msftclas
Copy link

msftclas commented Apr 13, 2018

CLA assistant check
All CLA requirements met.

@codecov
Copy link

codecov bot commented Apr 13, 2018

Codecov Report

Merging #1394 into master will increase coverage by 3.55%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1394      +/-   ##
==========================================
+ Coverage   71.01%   74.56%   +3.55%     
==========================================
  Files         269      285      +16     
  Lines       12416    13359     +943     
  Branches     2199     2390     +191     
==========================================
+ Hits         8817     9961    +1144     
+ Misses       3461     3265     -196     
+ Partials      138      133       -5
Impacted Files Coverage Δ
src/client/common/platform/pathUtils.ts 80% <0%> (-20%) ⬇️
src/client/unittests/main.ts 36.32% <0%> (-14.44%) ⬇️
...lient/debugger/configProviders/pythonV2Provider.ts 87.34% <0%> (-12.66%) ⬇️
src/client/common/platform/fileSystem.ts 67.53% <0%> (-11.64%) ⬇️
src/client/common/logger.ts 42.1% <0%> (-11.23%) ⬇️
src/client/interpreter/virtualEnvs/index.ts 89.36% <0%> (-10.64%) ⬇️
...ent/interpreter/locators/services/pipEnvService.ts 84.61% <0%> (-10.13%) ⬇️
src/client/interpreter/helpers.ts 88.37% <0%> (-7.09%) ⬇️
src/client/common/terminal/service.ts 93.75% <0%> (-6.25%) ⬇️
src/client/activation/classic.ts 91.83% <0%> (-5.67%) ⬇️
... and 125 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cd5095...869d9ef. Read the comment docs.

@brettcannon brettcannon changed the title [ADD] French Language French language translation Apr 16, 2018
@brettcannon
Copy link
Member

Thanks for the PR! Could you change the README and add a news entry file? (See #1152 for examples.)

I will also ask on Twitter for a review since none of our team members are native French speakers. 😄

"python.command.python.sortImports.title": "Trier les imports",
"python.command.python.startREPL.title": "Démarrer REPL",
"python.command.python.createTerminal.title": "Créer un Terminal",
"python.command.python.buildWorkspaceSymbols.title": "Build Workspace Symbols",

Choose a reason for hiding this comment

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

Construire les symboles de l'espace de travail ?

"python.command.python.execInTerminal.title": "Exécuter le script Python dans un Terminal",
"python.command.python.setInterpreter.title": "Selectionner l'interpreteur",
"python.command.python.updateSparkLibrary.title": "Mettre à jour les librairies de l'espace de travail pySpark",
"python.command.python.refactorExtractVariable.title": "Extraire la ariable",

Choose a reason for hiding this comment

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

Typo "variable"

"python.snippet.launch.flask.label": "Python: Flask (0.11.x ou supérieur)",
"python.snippet.launch.flask.description": "Debugger une Application Flask",
"python.snippet.launch.flaskOld.label": "Python: Flask (0.10.x ou antérieur)",
"python.snippet.launch.flaskOld.description": "Debug une application Flask de l'ancien style",

Choose a reason for hiding this comment

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

I think the French version is written "déboguer"

http://www.larousse.fr/dictionnaires/francais/d%C3%A9boguer/21835

Choose a reason for hiding this comment

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

yes, and débugger is also commonly used & accepted

@PixiBixi
Copy link
Author

Ahah @brettcannon its fine, i'll custom the README file :D

Thanks @browniebroke i'll fix it ASAP. And indeed, we can say déboguer in "pure" french, but tbh, everyone understand if i put "Debug" or "Debugger", but i'll fix it

Copy link

@tarekziade tarekziade left a comment

Choose a reason for hiding this comment

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

LGTM besides uppercases and a choice to make on how to write "to debug")

@@ -0,0 +1,49 @@
{
"python.command.python.sortImports.title": "Trier les imports",
"python.command.python.startREPL.title": "Démarrer REPL",

Choose a reason for hiding this comment

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

Démarrer la console interactive

(don't put uppercase words in sentences in French)

{
"python.command.python.sortImports.title": "Trier les imports",
"python.command.python.startREPL.title": "Démarrer REPL",
"python.command.python.createTerminal.title": "Créer un Terminal",

Choose a reason for hiding this comment

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

Créer un terminal

"python.command.python.createTerminal.title": "Créer un Terminal",
"python.command.python.buildWorkspaceSymbols.title": "Build Workspace Symbols",
"python.command.python.runtests.title": "Exécuter tous les tests unitaires",
"python.command.python.debugtests.title": "Debugger tous les tests unitaires",

Choose a reason for hiding this comment

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

Déboguer tous les tests unitaires

(notice that Débugger is also accepted)

"python.command.python.buildWorkspaceSymbols.title": "Build Workspace Symbols",
"python.command.python.runtests.title": "Exécuter tous les tests unitaires",
"python.command.python.debugtests.title": "Debugger tous les tests unitaires",
"python.command.python.execInTerminal.title": "Exécuter le script Python dans un Terminal",

Choose a reason for hiding this comment

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

s/Terminal/terminal

"python.command.python.runtests.title": "Exécuter tous les tests unitaires",
"python.command.python.debugtests.title": "Debugger tous les tests unitaires",
"python.command.python.execInTerminal.title": "Exécuter le script Python dans un Terminal",
"python.command.python.setInterpreter.title": "Selectionner l'interpreteur",

Choose a reason for hiding this comment

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

Sélectionner l'interpréteur

"python.snippet.launch.flaskOld.label": "Python: Flask (0.10.x ou antérieur)",
"python.snippet.launch.flaskOld.description": "Debug une application Flask de l'ancien style",
"python.snippet.launch.pyramid.label": "Python: application Pyramid",
"python.snippet.launch.pyramid.description": "Debugger une application Pyramid",

Choose a reason for hiding this comment

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

Débugger or Déboguer

"python.snippet.launch.pyramid.label": "Python: application Pyramid",
"python.snippet.launch.pyramid.description": "Debugger une application Pyramid",
"python.snippet.launch.watson.label": "Python: Application Watson",
"python.snippet.launch.watson.description": "Debugger une Application Watson",

Choose a reason for hiding this comment

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

Débugger une application Watson

"python.snippet.launch.watson.label": "Python: Application Watson",
"python.snippet.launch.watson.description": "Debugger une Application Watson",
"python.snippet.launch.attach.label": "Python: Attacher",
"python.snippet.launch.attach.description": "Attacher le debugger pour un debugging distant",

Choose a reason for hiding this comment

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

débogueur or débugueur (yeah I know, the second one is weird, but it's the French adaptation of the English word - if you picked Débugger earlier, use that one)

"python.snippet.launch.attach.label": "Python: Attacher",
"python.snippet.launch.attach.description": "Attacher le debugger pour un debugging distant",
"python.snippet.launch.scrapy.label": "Python: Scrapy",
"python.snippet.launch.scrapy.description": "Scrapy avec un Terminal intégré"

Choose a reason for hiding this comment

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

terminal

"python.snippet.launch.attach.description": "Attacher le debugger pour un debugging distant",
"python.snippet.launch.scrapy.label": "Python: Scrapy",
"python.snippet.launch.scrapy.description": "Scrapy avec un Terminal intégré"
}

Choose a reason for hiding this comment

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

no new line at EOF

Copy link
Author

Choose a reason for hiding this comment

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

Thats kinda better, thanks for your fix :)

@brettcannon
Copy link
Member

@PixiBixi FYI we go into code freeze for the next release on Monday, April 30, so if we can get the required changes in by then we can include this in the April release. 😄

@brettcannon
Copy link
Member

This has missed the April release. If @PixiBixi doesn't have time to address comments sometime this week then we will be open to someone forking this PR and making the appropriate changes along with listing @PixiBixi as a co-author on the PR.

@brettcannon
Copy link
Member

Since we aren't getting follow-up from @PixiBixi , anyone can feel free to fork this and submit a new PR with a French translation.

@PixiBixi
Copy link
Author

Ohhh i had absolutely forget this PR @brettcannon, sorry ! I'll try to fix it in the week :)

@brettcannon
Copy link
Member

@PixiBixi thanks!

@n6g7
Copy link

n6g7 commented Jun 9, 2018

@PixiBixi let me know if you need help, I can pick this up if need be.

@smortaz
Copy link

smortaz commented Jun 9, 2018 via email

@brettcannon
Copy link
Member

@huguesv feel free to provide feedback if you want (or a fr-ca translation 😁 ).

"python.snippet.launch.flask.label": "Python: Flask (0.11.x ou supérieur)",
"python.snippet.launch.flask.description": "Débugger une application Flask",
"python.snippet.launch.flaskOld.label": "Python: Flask (0.10.x ou antérieur)",
"python.snippet.launch.flaskOld.description": "Debug une application Flask de l'ancien style",

Choose a reason for hiding this comment

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

One last thing, I think the "Debug" here should be changed to "Débugger" to be consistent with the other wording.

Copy link

Choose a reason for hiding this comment

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

"de l'ancien style" is very vague, I think we should use "version" instead. Maybe something like "Débugger une application Flask (ancienne version)"?

Copy link
Member

Choose a reason for hiding this comment

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

Why not follow the label description:
Débugger une application Flask (0.10.x ou antérieur)

"python.snippet.launch.flask.label": "Python: Flask (0.11.x ou supérieur)",
"python.snippet.launch.flask.description": "Débugger une application Flask",
"python.snippet.launch.flaskOld.label": "Python: Flask (0.10.x ou antérieur)",
"python.snippet.launch.flaskOld.description": "Debug une application Flask de l'ancien style",
Copy link

Choose a reason for hiding this comment

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

"de l'ancien style" is very vague, I think we should use "version" instead. Maybe something like "Débugger une application Flask (ancienne version)"?

"python.command.python.runLinting.title": "Exécuter le linting",
"python.snippet.launch.standard.label": "Python: Fichier actuel",
"python.snippet.launch.standard.description": "Débugger un programme Python avec la sortie standard",
"python.snippet.launch.pyspark.label": "Python: PySpark",
Copy link

Choose a reason for hiding this comment

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

There's a space before the colon in French: "Python : PySpark". Same for all other labels below.

"python.snippet.launch.watson.label": "Python: Application Watson",
"python.snippet.launch.watson.description": "Débugger une Application Watson",
"python.snippet.launch.attach.label": "Python: Attacher",
"python.snippet.launch.attach.description": "Attacher le débugger pour un debugging distant",
Copy link

Choose a reason for hiding this comment

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

The noun "débugger" (debugger) is spelt exactly the same as the verb "débugger" (to debug), is there a different spelling we could use to make it clearer?

Copy link

Choose a reason for hiding this comment

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

VSCode itself actually uses "déboguer" and "débogueur", perhaps we should use that instead for consistence?
image

Copy link
Member

Choose a reason for hiding this comment

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

I agree for consistency, if general VSCode is already doing "Déboguer/débogage/débogueurs", we should use it here as well.

@brettcannon brettcannon merged commit 5949f74 into microsoft:master Jun 14, 2018
@brettcannon
Copy link
Member

I have gone ahead and merged this with the latest round of comments. If there is anything people want changed before the next release (scheduled for Monday), please send a PR. I will send out a separate PR to add the translation to the README and the news entry.

Thanks to everyone who helped with this!

@lock lock bot locked as resolved and limited conversation to collaborators Jul 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants