node-integration-tests: drop static pins from non-tracing suites
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 74/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- node.js, typescript
- 領域
- testing-qa
調査の方向性
親 issue の移植手順から始め、その後、記載されている node-integration-tests の suite ディレクトリで traceLifecycle: 'static' を検索します。“pin removal only” に記載されている tracing なしの suite から pin を削除し、7 つの assertion 書き換えグループは別の作業として残します。これらの suite に不要な static pin が残っておらず、記載されている envelope を assertion する suite が変更されていなければ完了です。
索引モデルが issue の本文から書いたものです。
説明
Follow the porting recipe in the parent issue.
Largest group by file count and the smallest by effort. Roughly 90 suite directories carry the traceLifecycle: 'static' pin without ever asserting on a transaction envelope, so the pin just needs to go.
Suites (pin removal only)
public-api/*except the span surface dirs owned by the trace semantics sub-issue:addBreadcrumb,captureException,captureMessage,configureScope,LocalVariables,logger,metrics,OnUncaughtException,setContext,setExtra,setExtras,setTag,setTags,setUser,withMonitor,withScopesessions,cron/*,client-reports/periodic-send- Logging:
pino,winston,consola,integrations/console/filter - Runtime:
anr,system-error,thread-blocked-native,child-process,modules,esm/*,contextLines/*,breadcrumbs/process-thread,node-runtime-metrics,bun-runtime-metrics vercel/sigterm-flush
Suites that still need an assertion rewrite
These 7 do assert on transaction envelopes:
public-api/onUnhandledRejectionIntegrationclient-reports/drop-reasons/*(4 dirs)featureFlags/featureFlagsIntegration/onSpanandfeatureFlags/growthbook/onSpanfs-instrumentationaws-serverless/aws-integration(foldaws-serverless/aws-integration-streamedback into it)
Splitting this issue into a mechanical pin-removal PR and a smaller rewrite PR is fine.
- 主要言語
- TypeScript
- スター
- 8.7k
- フォーク
- 1.9k
- 平均マージ
- 1日 18時間
- マージ済み PR（30日）
- 562
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
getsentry/sentry-javascript のほかの issue
-
Browser Waiting for: Product Owner
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
getsentry/sentry-javascript#24577 · コメント 1 件 ·
-
Task
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
getsentry/sentry-javascript#24558 · コメント 1 件 ·
-
Task
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
getsentry/sentry-javascript#24557 · コメント 1 件 ·
-
Task
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
getsentry/sentry-javascript#24556 · コメント 1 件 ·
-
Task
難易度 1/5 1〜3時間 初心者へのやさしさ 90/100
getsentry/sentry-javascript#24555 · コメント 1 件 ·
getsentry/sentry-javascript の issue をすべて見る
似ている issue
-
blocklist removal
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
MetaMask/eth-phishing-detect#296544 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
pastelsky/bundlephobia#1122 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100