-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[pt] Localize content/en/docs/languages/js/instrumentation.md #8388
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
base: main
Are you sure you want to change the base?
[pt] Localize content/en/docs/languages/js/instrumentation.md #8388
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.
Pull Request Overview
This PR localizes the JavaScript instrumentation documentation from English to Portuguese (pt), providing comprehensive guidance on manual instrumentation for OpenTelemetry JavaScript.
- Translates the complete instrumentation guide including traces, metrics, and logs sections
- Provides Portuguese translations of code comments and technical terms
- Maintains the document structure and all code examples from the English version
Co-authored-by: Copilot <[email protected]>
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.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
| const sdk = new NodeSDK({ | ||
| resource: resourceFromAttributes({ | ||
| [ATTR_SERVICE_NAME]: 'nomeDoServico', | ||
| [ATTR_SERVICE_VERSION]: '1.0', |
Copilot
AI
Nov 11, 2025
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.
The TypeScript example uses version '1.0' while the JavaScript example at line 264 uses '0.1.0'. These should match for consistency, preferably using '0.1.0' to match the rest of the examples in the document (lines 374, 410, 512, 541, etc.).
| [ATTR_SERVICE_VERSION]: '1.0', | |
| [ATTR_SERVICE_VERSION]: '0.1.0', |
|
|
||
| const metricReader = new PeriodicExportingMetricReader({ | ||
| exporter: new ConsoleMetricExporter(), | ||
| // O valor padrão é 60000ms (60 segundos). Vamos utilizar 10 segundos apenas para fins de demonstração. |
Copilot
AI
Nov 11, 2025
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.
[nitpick] Corrected 'Vamos' to 'Usando' or 'Utilizado' for consistency with line 1383 which uses similar phrasing.
| // O valor padrão é 60000ms (60 segundos). Vamos utilizar 10 segundos apenas para fins de demonstração. | |
| // O valor padrão é 60000ms (60 segundos). Utilizando 10 segundos apenas para fins de demonstração. |
Description
content/pt/docs/languages/js/instrumentation.mdPreview: https://deploy-preview-8388--opentelemetry.netlify.app/