-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 localization: content/pt/docs/concepts/signals/baggage.md #5188
[pt] add localization: content/pt/docs/concepts/signals/baggage.md #5188
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI: Olhando a versão em inglês, lá estava inconsistente quando usar Bagagem vs bagagem, então tinha adicionado uns comentários aqui pra deixar tudo como bagagem, já que normalmente não capitaliza outra palavras do mesmo estilo (trace, span, metrics, logs, ...), mas olhando na página de especificação lá está consistentemente como Bagagem, então apaguei os comentários relacionados a isso 😄
Fora isso, só alguns nits
Co-authored-by: Marylia Gutierrez <maryliag@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ficou muito bom!
/fix:all |
You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/10904389811 |
fix:all failed or was cancelled. For details, see https://github.com/open-telemetry/opentelemetry.io/actions/runs/10904389811. |
Adds Baggage location, update this PR that has not been updated for a time.
Issue: #4922
PR: https://github.com/open-telemetry/opentelemetry.io/pull/4993/files