`getDefaultIntegrations` no longer extends the base set

Aberta Para iniciantes
#24,039 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
84/100
Tipo de issue
Documentação
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
documentation

Direção de pesquisa

Leia packages/cloudflare/src/baseSdk.ts:45, sdk.ts:13 e request.ts:9 e, em seguida, compare os conjuntos de integrações padrão e o histórico relacionado descrito na issue. Atualize os comentários para que descrevam corretamente os conjuntos idênticos e remova qualquer recomendação que não tenha efeito; confirme que a documentação está consistente e que nenhuma funcionalidade foi alterada.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Cloudflare Workers javascript

packages/cloudflare/src/baseSdk.ts:45 documents that "getDefaultIntegrations in sdk.ts extends this set with the integrations that do depend on Node.js APIs". sdk.ts:13 is:

export function getDefaultIntegrations(options: CloudflareOptions): Integration[] {
  return getBaseDefaultIntegrations(options);
}

The two sets are identical. packages/cloudflare/src/request.ts:9 then tells users to "pass defaultIntegrations: getDefaultIntegrations(options) in options to get the full set instead", which is now a no-op.

Work item. Work out whether a nodejs_compat-only integration was dropped or never added. Then either restore it or correct both doc comments. Do not leave advice in the docs that does nothing.

**Prior art **(related history only). No issue, but the cause is clear from the history. getsentry/sentry-javascript#21881 added a separate /nodejs_compat entrypoint with the larger integration set, and getsentry/sentry-javascript#22900 "Make the nodejs_compat entrypoint the default" folded it into the default entrypoint and removed it, closing #18803. The two sets became identical at that point and the doc comments were not updated. This is documentation cleanup, not lost functionality.

Linguagem predominante
TypeScript
Estrelas
8.7k
Forks
1.9k
Merge médio
1d 17h
PRs com merge (30d)
521

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de getsentry/sentry-javascript

Todas as issues de getsentry/sentry-javascript

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.