-
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] Translate concepts/instrumentation-scope #5604
base: main
Are you sure you want to change the base?
[pt] Translate concepts/instrumentation-scope #5604
Conversation
7e4d0ee
to
cf8dfc4
Compare
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.
LGTM!!!
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
Co-authored-by: Marylia Gutierrez <[email protected]>
@maryliag perfeito! obrigado pelas sugestões 🚀 |
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.
Eu que agradeço sua contribuição!
LGTM
/fix:all |
You triggered fix:all action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/11975457924 |
IMPORTANT: (RE-)RUN
|
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.
@chalin I think images do not need to be copied, right?
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.
Yep, it makes sense to keep the original source!
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.
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.
as far as I know if you leave it out it will be picked from the en
version but I am not 100% sure about that. @chalin knows best but is out of office until early next week, let's give him some time to take a look and respond
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.
Sure! No worries. Thanks @svrnm :)
Fixes #4922