`getDefaultIntegrations` no longer extends the base set

Đang mở Phù hợp với người mới
#24,039 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
84/100
Loại issue
Tài liệu
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
typescript
Lĩnh vực
documentation

Hướng nghiên cứu

Đọc packages/cloudflare/src/baseSdk.ts:45, sdk.ts:13 và request.ts:9, sau đó so sánh các tập hợp tích hợp mặc định và lịch sử liên quan được mô tả trong issue. Cập nhật các comment để mô tả chính xác các tập hợp giống hệt nhau và loại bỏ mọi hướng dẫn không có tác dụng; xác nhận rằng tài liệu nhất quán và không có chức năng nào bị thay đổi.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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.

Ngôn ngữ chính
TypeScript
Star
8.7k
Fork
1.9k
Merge trung bình
1 ngày 17 giờ
Pull request đã merge (30 ngày)
521

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của getsentry/sentry-javascript

Tất cả issue của getsentry/sentry-javascript

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.