WIP Initial Portuguese Brazil translation. #13572

jcjesus · 2019-04-02T13:36:08Z

WIP - Initial Portuguese Brazil translation.

k8s-ci-robot · 2019-04-02T13:36:21Z

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
If you have done the above and are still having issues with the CLA being reported as unsigned, please email the CNCF helpdesk: [email protected]

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

k8s-ci-robot · 2019-04-02T13:36:28Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: kbarnard10

If they are not already assigned, you can assign the PR to them by writing /assign @kbarnard10 in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

remyleone · 2019-04-02T13:56:48Z

/checkcla

remyleone · 2019-04-02T14:00:48Z

You need to add a i18n/pt.toml file you can take example on the english one

remyleone · 2019-04-02T15:24:10Z

OWNERS_ALIASES

    - cstoku
    - nasa9084
    - tnir
    - zacharysarah
-  sig-docs-ja-reviews: # PR reviews for Italian content 
+  sig-docs-ja-reviews: # PR reviews for Italian content


Beware there is some whitespaces changes here

remyleone · 2019-04-02T15:27:21Z

I think you can use only pt as it is the ISO 639-1 standard https://www.loc.gov/standards/iso639-2/php/code_list.php

remyleone · 2019-04-02T16:24:15Z

i18n/pt-BR.toml

@@ -0,0 +1,139 @@
+# i18n strings for the English (main) site.


I think you can remove this comment

remyleone · 2019-04-02T16:24:59Z

config.toml

+
+[languages.pt-BR]
+title = "Kubernetes"
+description = "Production-Grade Container Orchestration"


Feel free to translate it to Portuguese as well :)

remyleone · 2019-04-02T16:25:42Z

README-pt-BR.md

+make serve
+```
+
+This will start the local Hugo server on port 1313. Open up your browser to http://localhost:1313 to view the site. As you make changes to the source files, Hugo updates the site and forces a browser refresh.


There is still English in this file

zacharysarah · 2019-04-02T19:26:28Z

@jcjesus Thanks for beginning this localization!

@sieben Thank you for the helpful reviews. 🙇🏼‍♂️🌟

zacharysarah · 2019-04-02T20:39:50Z

/check-cla

zacharysarah · 2019-04-02T20:44:51Z

@jcjesus 👋 It looks like your CLA check is failing because the commits are attached to a different GitHub ID, julio.jesus. You can resolve it by rebasing all of your commits from @jcjesus.

femrtnz · 2019-04-03T06:00:28Z

i18n/pt-BR.toml

+# i18n strings for the English (main) site.
+
+[deprecation_warning]
+other = " documentação não será manutenida por mais tempo. A versão que você está visualizando atualmente é um versão desatualizada e estática. Para obter documentação atualizada, consulte "


Maybe mantida instead of manutenida ?

Yes, greate @femrtnz!

femrtnz · 2019-04-03T06:06:03Z

content/pt-BR/OWNERS

@@ -0,0 +1,13 @@
+# See the OWNERS docs at https://go.k8s.io/owners
+
+# This is the localization project for Portuguese Brazils.


Wondering if we need to translate comments like that...

femrtnz · 2019-04-11T12:50:28Z

content/pt-BR/OWNERS

@@ -0,0 +1,13 @@
+# See the OWNERS docs at https://go.k8s.io/owners


I think we should change the folder to "pt" only, following the same folder structure as the older content.
e.g:
fr
en

zacharysarah · 2019-04-16T20:17:19Z

Closing in favor of #13773.

Initial Portuguese Brazil translation.

2c7228b

k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 2, 2019

k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Apr 2, 2019

k8s-ci-robot requested review from xiangpengzhao and zhangxiaoyu-zidif April 2, 2019 13:36

remyleone self-assigned this Apr 2, 2019

jesus-amido added 2 commits April 2, 2019 11:15

WIP adding Owners and Reviewers.

09a78ed

WIP adding pt-BR.toml file.

6d73580

remyleone reviewed Apr 2, 2019

View reviewed changes

Remove space between owner and reviewer.

f652425

remyleone reviewed Apr 2, 2019

View reviewed changes

i18n/pt-BR.toml

@@ -0,0 +1,139 @@

# i18n strings for the English (main) site.

Copy link

Contributor

remyleone Apr 2, 2019

Choose a reason for hiding this comment

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

I think you can remove this comment

remyleone reviewed Apr 2, 2019

View reviewed changes

femrtnz requested changes Apr 3, 2019

View reviewed changes

femrtnz reviewed Apr 11, 2019

View reviewed changes

zacharysarah closed this Apr 16, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

WIP Initial Portuguese Brazil translation. #13572

WIP Initial Portuguese Brazil translation. #13572

jcjesus commented Apr 2, 2019

k8s-ci-robot commented Apr 2, 2019

k8s-ci-robot commented Apr 2, 2019

remyleone commented Apr 2, 2019

remyleone commented Apr 2, 2019

remyleone Apr 2, 2019

remyleone commented Apr 2, 2019 •

edited

Loading

remyleone Apr 2, 2019

remyleone Apr 2, 2019

remyleone Apr 2, 2019

zacharysarah commented Apr 2, 2019

zacharysarah commented Apr 2, 2019

zacharysarah commented Apr 2, 2019

femrtnz Apr 3, 2019

jcjesus Apr 3, 2019

femrtnz Apr 3, 2019

femrtnz Apr 11, 2019

zacharysarah commented Apr 16, 2019

		@@ -0,0 +1,139 @@
		# i18n strings for the English (main) site.

		@@ -0,0 +1,13 @@
		# See the OWNERS docs at https://go.k8s.io/owners

		# This is the localization project for Portuguese Brazils.

WIP Initial Portuguese Brazil translation. #13572

WIP Initial Portuguese Brazil translation. #13572

Conversation

jcjesus commented Apr 2, 2019

k8s-ci-robot commented Apr 2, 2019

k8s-ci-robot commented Apr 2, 2019

remyleone commented Apr 2, 2019

remyleone commented Apr 2, 2019

Choose a reason for hiding this comment

remyleone commented Apr 2, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

zacharysarah commented Apr 2, 2019

zacharysarah commented Apr 2, 2019

zacharysarah commented Apr 2, 2019

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

zacharysarah commented Apr 16, 2019

remyleone commented Apr 2, 2019 •

edited

Loading