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

[pt-br] Localize docs/concepts/windows/intro #48932

Open
wants to merge 24 commits into
base: main
Choose a base branch
from

Conversation

rafaelmaferreira
Copy link
Contributor

@rafaelmaferreira rafaelmaferreira commented Dec 2, 2024

Description:

Creating /pt-br/docs/concepts/windows/intro

Closes: #48930

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 2, 2024
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign edsoncelio for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

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

@k8s-ci-robot k8s-ci-robot added the area/localization General issues or PRs related to localization label Dec 2, 2024
@k8s-ci-robot k8s-ci-robot added language/pt Issues or PRs related to Portuguese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Dec 2, 2024
@rafaelmaferreira
Copy link
Contributor Author

@edsoncelio @knabben

@rafaelmaferreira rafaelmaferreira changed the title [pt-br] docs/concepts/windows/intro/ [pt-br] Localize docs/concepts/windows/intro Dec 2, 2024
Copy link

netlify bot commented Dec 2, 2024

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit b92917e
🔍 Latest deploy log https://app.netlify.com/sites/kubernetes-io-main-staging/deploys/6758d9874dd6b00008b94e62
😎 Deploy Preview https://deploy-preview-48932--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@stormqueen1990
Copy link
Member

stormqueen1990 commented Dec 2, 2024 via email

Copy link
Member

@stormqueen1990 stormqueen1990 left a comment

Choose a reason for hiding this comment

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

Olá @rafaelmaferreira! Muito obrigada pela tua contribuição 🙏🏻

Textualmente, a localização está bem encaminhada. Peço perdão pela quantidade de comentários, mas a maioria deles são ajustes pontuais para padronização do texto.

Alguns itens de atenção:

  • Percebi o uso de "Windows OS", mas em português seria mais adequado usar "SO Windows" ou "sistema operacional Windows". A escolha entre um desses dois fica a teu critério.
  • Na localização em português, temos alguns termos já padronizados. Por isso, peço que ajustes os usos dos termos abaixo:
    • Contêiner/contêineres no lugar de container/containers.
    • Agente de execução de contêiner no lugar de container runtime.
    • Camada de gerenciamento no lugar de control plane/plano de controle.
  • Notei diferenças de formatação de texto entre a página original e a localização. Como via de regra, devemos manter a formatação. Existe algum motivo específico para que isso tenha sido alterado?

Caso tenhas quaisquer dúvidas ou discordes de qualquer parte dessa revisão, não hesita em me contatar no canal #kubernetes-docs-pt do Slack do Kubernetes. Assim que os ajustes forem efetuados podes me solicitar uma nova revisão nesta página.

Muito obrigada novamente!

Comment on lines 2 to 6
reviewers:
- jayunit100
- jsturtevant
- marosset
- perithompson
Copy link
Member

Choose a reason for hiding this comment

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

Este bloco pode ser removido totalmente.

Suggested change
reviewers:
- jayunit100
- jsturtevant
- marosset
- perithompson
reviewers:
- jayunit100
- jsturtevant
- marosset
- perithompson

content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
content/pt-br/docs/concepts/windows/intro.md Outdated Show resolved Hide resolved
@stormqueen1990
Copy link
Member

/label tide/merge-method-squash

@k8s-ci-robot k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Dec 3, 2024
rafaelmaferreira and others added 5 commits December 4, 2024 20:48
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
rafaelmaferreira and others added 11 commits December 4, 2024 20:50
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
Co-authored-by: Mauren <698465+stormqueen1990@users.noreply.github.com>
@rafaelmaferreira
Copy link
Contributor Author

Ola @stormqueen1990 Muito obrigado pelas revisões e meus parabéns por todo trabalho que tem feito nas docs!

Corrigi as menções, poderia por favor verificar novamente?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/localization General issues or PRs related to localization cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/pt Issues or PRs related to Portuguese language sig/docs Categorizes an issue or PR as relevant to SIG Docs. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[pt-br] Localize docs/concepts/windows/intro
3 participants