Skip to content

Conversation

@vitorvasc
Copy link
Member

@vitorvasc vitorvasc commented Nov 11, 2025

Description

Preview: https://deploy-preview-8388--opentelemetry.netlify.app/

@vitorvasc vitorvasc requested a review from Copilot November 11, 2025 12:22
Copilot finished reviewing on behalf of vitorvasc November 11, 2025 12:25
Copy link
Contributor

Copilot AI left a 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

@vitorvasc vitorvasc requested a review from Copilot November 11, 2025 13:18
Copilot finished reviewing on behalf of vitorvasc November 11, 2025 13:19
Copy link
Contributor

Copilot AI left a 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',
Copy link

Copilot AI Nov 11, 2025

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.).

Suggested change
[ATTR_SERVICE_VERSION]: '1.0',
[ATTR_SERVICE_VERSION]: '0.1.0',

Copilot uses AI. Check for mistakes.

const metricReader = new PeriodicExportingMetricReader({
exporter: new ConsoleMetricExporter(),
// O valor padrão é 60000ms (60 segundos). Vamos utilizar 10 segundos apenas para fins de demonstração.
Copy link

Copilot AI Nov 11, 2025

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.

Suggested change
// 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.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[pt] Localize content/en/docs/languages/js/instrumentation.md

1 participant