Add some PT-BR translations #3102

cpanato · 2017-10-20T14:02:32Z

Proposed changelog entries

Add some PT-BR translations

cpanato · 2017-10-20T14:02:41Z

@jenkinsci/hacktoberfest

daniel-beck · 2017-10-20T14:04:25Z

@oleg-nenashev Apparently team mentions don't work for people outside the organization.

daniel-beck · 2017-10-20T14:05:15Z

@kinow Is this something you could review?

cpanato · 2017-10-20T14:05:15Z

@daniel-beck :( I saw this https://jenkins.io/blog/2017/10/06/hacktoberfest/

cpanato · 2017-10-20T14:05:56Z

@daniel-beck I will add more during next week and so on :)

daniel-beck · 2017-10-20T14:06:05Z

@cpanato Thanks, I know, that's why I'm telling Oleg this might not work as expected. I edited your comment so the mention works, but that doesn't scale…

cpanato · 2017-10-20T16:39:04Z

@daniel-beck looks like the mention trigger did not work as well.
and regarding the build, I saw all steps were green, what is the issue?

kinow

Translating from English to Portuguese is really tricky. Found a few minor typos, and some sentences that could be re-written to sound more natural. Though that's still a bit subjective.

@boaglio might be able to help too. He has experience with i18n on KDE, and did most of the pt_BR for Jenkins.

kinow · 2017-10-20T20:30:30Z

core/src/main/resources/hudson/Messages_pt_BR.properties

+# {0} v{1} is missing. To fix, install v{1} or later.
+PluginWrapper.missing={0} v{1} est\u00E1 faltando. Para corrigir, instale v{1} ou superior.
+# No Exception details
+Functions.NoExceptionDetails=Sem detalhes da exce\u00E7\u00E3o


Exception is normally left untranslated

kinow · 2017-10-20T20:32:04Z

core/src/main/resources/hudson/Messages_pt_BR.properties

@@ -76,3 +76,38 @@ ProxyConfiguration.FailedToConnect=Falha ao conectar em {0} (c\u00f3digo {1}).
 ProxyConfiguration.Success=Sucesso
 # Did not manage to validate {0} (may be too slow)
 FilePath.did_not_manage_to_validate_may_be_too_sl=N\u00e3o conseguiu validar {0} (pode ser muito lento)
+# {0} v{1} is missing. To fix, install v{1} or later.
+PluginWrapper.missing={0} v{1} est\u00E1 faltando. Para corrigir, instale v{1} ou superior.


Maybe something like "Não foi possível encontrar {0} v{1}" would sound better

kinow · 2017-10-20T20:33:41Z

core/src/main/resources/hudson/Messages_pt_BR.properties

+# Plugins Failed To Load
+PluginWrapper.PluginWrapperAdministrativeMonitor.DisplayName=Falha ao carregar o plugin
+# Malformed Test URL {0}.
+ProxyConfiguration.MalformedTestUrl=URL de test mal formatada {0}.


In this case it is better to translate test to "teste". But something like "URL de teste inválida" would probably sound better

kinow · 2017-10-20T20:35:32Z

core/src/main/resources/hudson/Messages_pt_BR.properties

+Functions.NoExceptionDetails=Sem detalhes da exce\u00E7\u00E3o
+# \ 
+# Whitespace can no longer be used as the separator. Please Use \u2018,\u2019 as the separator instead.
+FilePath.validateAntFileMask.whitespaceSeparator=Espa\u00E7o em branco n\u00E3o podem mais serrem utilizados como separador. Por favor use  \u2018, \u2019 como separadores.


There is a lack of concordance here. It should be in the plural, "Espa\u00E7os".

There is also a typo, as instead of "serrem", it has to be "serem"

kinow · 2017-10-20T20:37:12Z

core/src/main/resources/hudson/Messages_pt_BR.properties

+# Whitespace can no longer be used as the separator. Please Use \u2018,\u2019 as the separator instead.
+FilePath.validateAntFileMask.whitespaceSeparator=Espa\u00E7o em branco n\u00E3o podem mais serrem utilizados como separador. Por favor use  \u2018, \u2019 como separadores.
+# Ping protocol
+TcpSlaveAgentListener.PingAgentProtocol.displayName=Protocolo de ping


This one sounds a bit odd in Portuguese, but I can't think of a better way. Probably because I never heard it, but should be OK

I will leave as is, or we can keep the original

kinow · 2017-10-20T21:11:42Z

core/src/main/resources/hudson/model/Computer/sidepanel_pt_BR.properties

@@ -26,3 +26,4 @@ Configure=Configurar
 Status=Situa\u00e7\u00e3o
 Load\ Statistics=Estat\u00edsticas de carga
 Script\ Console=Terminal de script
+Delete\ Agent=Apagar o agente


"Remover" would be better IMO.

kinow · 2017-10-20T21:12:22Z

core/src/main/resources/hudson/model/Job/configure_pt_BR.properties

@@ -25,3 +25,5 @@ name=Nome do {0}
 Description=Descri\u00e7\u00e3o
 Save=Salvar
 LOADING=CARREGANDO
+# Apply
+Apply=Aplicar


That's an interesting one. In other systems I normally see "Concluir", but I guess in other parts we are probably using Aplicar. So +1

kinow · 2017-10-20T21:15:57Z

core/src/main/resources/hudson/model/AbstractBuild/index_pt_BR.properties

@@ -33,3 +33,4 @@ Downstream\ Builds=builds filho
 none=nenhum
 Took=Demorou
 Build=Build
+on=ativo


In this case this translation would be incorrect. Here's where it is used: https://github.com/cpanato/jenkins/blob/2f11dc2fea19c65b0b9a399531ddf775bbe2e55a/core/src/main/resources/hudson/model/AbstractBuild/index.jelly#L45

The "on" is used in a sentence like "Took 10ms on ...."

Probably "em" would be a better translation.

kinow · 2017-10-20T21:16:50Z

core/src/main/resources/hudson/Messages_pt_BR.properties

+# \u2018{0}\u2019 doesn\u2019t match anything: even \u2018{1}\u2019 doesn\u2019t exist
+FilePath.validateAntFileMask.doesntMatchAnythingAndSuggest=\u2018{0}\u2019 n\u00E3o combina nada: o   \u2018{1}\u2019 n\u00E3o existe
+# {0} v{1} failed to load. Fix this plugin first.
+PluginWrapper.failed_to_load_dependency={0} v{1} falhou ao carregar. Primeiro corrija este plugin.


"Primeiro corrija este plugin." would be better re-worded as "Corrija o problema com este plugin primeiro" IMO.

kinow · 2017-10-20T21:19:36Z

core/src/main/resources/hudson/Messages_pt_BR.properties

+# You must update Jenkins from v{0} to v{1} or later to run this plugin.
+PluginWrapper.obsoleteCore=Voc\u00EA deve atualizar o Jenkins de v{0} para v{1} ou superior para executar este plugin.
+# \u2018{0}\u2019 doesn\u2019t match anything: even \u2018{1}\u2019 doesn\u2019t exist
+FilePath.validateAntFileMask.doesntMatchAnythingAndSuggest=\u2018{0}\u2019 n\u00E3o combina nada: o   \u2018{1}\u2019 n\u00E3o existe


Extra white spaces, but also the "combina" sounds a bit weird. It is used in the context that a ant file mask, a kind of glob, didn't find any matching entry. So probably "(...) não retornou nenhum resultado: nem mesmo ... existe"

cpanato · 2017-10-21T09:09:34Z

@kinow thanks for your review so much! I've pushed the fixes.

kinow

Minor nit pick. Other than that, looking good for me.

kinow · 2017-10-21T10:49:34Z

core/src/main/resources/hudson/Messages_pt_BR.properties

 # {0} v{1} is older than required. To fix, install v{2} or later.
 PluginWrapper.obsolete={0} v{1} \u00E9 mais antigo do que o requerido. Para corrigir, instale v{2} ou superior.
 # \u2018{0}\u2019 doesn\u2019t match anything because it is treated case sensitively. You can deactivate case sensitivity to get matches
 FilePath.validateAntFileMask.matchWithCaseInsensitive=
 # You must update Jenkins from v{0} to v{1} or later to run this plugin.
 PluginWrapper.obsoleteCore=Voc\u00EA deve atualizar o Jenkins de v{0} para v{1} ou superior para executar este plugin.
 # \u2018{0}\u2019 doesn\u2019t match anything: even \u2018{1}\u2019 doesn\u2019t exist
-FilePath.validateAntFileMask.doesntMatchAnythingAndSuggest=\u2018{0}\u2019 n\u00E3o combina nada: o   \u2018{1}\u2019 n\u00E3o existe
+FilePath.validateAntFileMask.doesntMatchAnythingAndSuggest=\u2018{0}\u2019 n\u00E3o retornou nenhum resultado: nem mesmo   \u2018{1}\u2019 existe


Still an extra space here"nem mesmo \u2018{1}\u2019 existe" after "mesmo".

daniel-beck · 2017-10-21T21:16:22Z

core/src/main/resources/hudson/model/AbstractBuild/changes_pt_BR.properties

@@ -21,3 +21,6 @@
 # THE SOFTWARE.

 Changes=Mudan\u00e7as
+Not\ yet\ determined=
+Failed\ to\ determine=


Empty values

oleg-nenashev

I see no issues in property file formatting, etc. Have not tested the content though

oleg-nenashev · 2017-10-26T23:20:53Z

@daniel-beck PTAL

boaglio · 2017-10-27T09:56:59Z

thanks @kinow and @cpanato for this =D

daniel-beck

No empty keys anymore AFAICT, so 🤷‍♂️

@oleg-nenashev forced me to do this.

oleg-nenashev · 2017-10-28T21:10:44Z

@cpanato thanks!

kinow suggested changes Oct 20, 2017

View reviewed changes

oleg-nenashev added the hacktoberfest Related to Hacktoberfest. https://hacktoberfest.digitalocean.com/ label Oct 21, 2017

kinow suggested changes Oct 21, 2017

View reviewed changes

Add some PT-BR translations

27defba

kinow approved these changes Oct 21, 2017

View reviewed changes

daniel-beck requested changes Oct 21, 2017

View reviewed changes

udpate strings per review

75f25e3

oleg-nenashev approved these changes Oct 25, 2017

View reviewed changes

daniel-beck approved these changes Oct 28, 2017

View reviewed changes

oleg-nenashev merged commit 1b4f875 into jenkinsci:master Oct 28, 2017

cpanato deleted the translate-pt-br branch October 29, 2017 08:23

Add some PT-BR translations #3102

Add some PT-BR translations #3102

Conversation

cpanato commented Oct 20, 2017

Proposed changelog entries

cpanato commented Oct 20, 2017 • edited by daniel-beck

daniel-beck commented Oct 20, 2017

daniel-beck commented Oct 20, 2017

cpanato commented Oct 20, 2017

cpanato commented Oct 20, 2017

daniel-beck commented Oct 20, 2017

cpanato commented Oct 20, 2017

kinow left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kinow Oct 20, 2017 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

cpanato commented Oct 21, 2017

kinow left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oleg-nenashev left a comment

Choose a reason for hiding this comment

oleg-nenashev commented Oct 26, 2017

boaglio commented Oct 27, 2017

daniel-beck left a comment

Choose a reason for hiding this comment

oleg-nenashev commented Oct 28, 2017

cpanato commented Oct 20, 2017 •

edited by daniel-beck

kinow Oct 20, 2017 •

edited