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] Add pt/content/docs/getting-started #4980

Merged
merged 8 commits into from
Aug 12, 2024

Conversation

marcelobaptista
Copy link
Contributor

Add a localization to the getting-started docs

Issue: #4922

@marcelobaptista marcelobaptista requested a review from a team August 6, 2024 22:34
Copy link

linux-foundation-easycla bot commented Aug 6, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@chalin
Copy link
Contributor

chalin commented Aug 6, 2024

/cc @edsoncelio @EzzioMoreira

@emdneto
Copy link
Member

emdneto commented Aug 7, 2024

/fix:all

@opentelemetrybot
Copy link
Collaborator

@opentelemetrybot
Copy link
Collaborator

fix:all run failed, please check https://github.com/open-telemetry/opentelemetry.io/actions/runs/10288903418 for details

Copy link
Member

@edsoncelio edsoncelio left a comment

Choose a reason for hiding this comment

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

Valeu pela contribuição @marcelobaptista!
Deixei alguns comentários, em caso de dúvida só falar!

content/pt/docs/getting-started/_index.md Outdated Show resolved Hide resolved
content/pt/docs/getting-started/_index.md Outdated Show resolved Hide resolved
content/pt/docs/getting-started/_index.md Outdated Show resolved Hide resolved
content/pt/docs/getting-started/_index.md Outdated Show resolved Hide resolved
content/pt/docs/getting-started/dev.md Show resolved Hide resolved
content/pt/docs/getting-started/ops.md Outdated Show resolved Hide resolved
content/pt/docs/getting-started/ops.md Outdated Show resolved Hide resolved
@EzzioMoreira
Copy link
Contributor

LGTM ✅

@svrnm
Copy link
Member

svrnm commented Aug 12, 2024

LGTM ✅

Will take this as an approval from the docs-pt-approvers since that group still does not exist

@svrnm
Copy link
Member

svrnm commented Aug 12, 2024

/fix:all

@opentelemetrybot
Copy link
Collaborator

@opentelemetrybot
Copy link
Collaborator

fix:all run failed, please check https://github.com/open-telemetry/opentelemetry.io/actions/runs/10348459952 for details

Comment on lines 2 to 3
título: Primeiros Passos
descrição: Comece a utilizar o OpenTelemetry com base na sua função.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
título: Primeiros Passos
descrição: Comece a utilizar o OpenTelemetry com base na sua função.
título: Primeiros Passos
descrição: Comece a utilizar o OpenTelemetry com base na sua função.

FYI, do not translate they keys here, this will break the page build + linters.

I will apply a fix commit and then it should look good.

Signed-off-by: svrnm <neumanns@cisco.com>
@svrnm svrnm added this pull request to the merge queue Aug 12, 2024
Merged via the queue into open-telemetry:main with commit fc5004d Aug 12, 2024
17 checks passed
@marcelobaptista marcelobaptista deleted the add_ptbr_getting-started branch August 12, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

7 participants