deslop a bit

Author: Lms24

dev-packages/node-integration-tests/suites/tracing/sampling-static/test.ts

@@ -9,8 +9,8 @@ describe('negative sampling (static)', () => {
   createEsmAndCjsTests(__dirname, 'server.mjs', 'instrument.mjs', (createRunner, test) => {
     test('records sample_rate outcomes for the transaction and all of its spans', async () => {
       // `/health` and `/ok` go through the same middleware, so the `/ok` transaction tells us how many
-      // spans the dropped `/health` transaction had. The count differs per runtime (e.g. Bun creates
-      // no Express spans), so derive it instead of hardcoding it.
+      // spans the dropped `/health` transaction woudl have had. The count differs per runtime (e.g. Bun creates
+      // no Express spans), so we derive it instead of hardcoding it.
       let okSpanCount: number | undefined;
       let droppedSpanCount: number | undefined;
 

packages/core/src/tracing/sentrySpan.ts

@@ -369,7 +369,6 @@ export class SentrySpan implements Span {
         return;
       }
 
-      // The `sample_rate` outcome was already recorded when the span was started (see `_startRootSpan`).
       DEBUG_BUILD && debug.log('[Tracing] Discarding standalone span because its trace was not chosen to be sampled.');
       return;
     }

packages/core/src/tracing/trace.ts

@@ -525,7 +525,7 @@ function _startRootSpan(
     DEBUG_BUILD && debug.log('[Tracing] Discarding root span because its trace was not chosen to be sampled.');
     const outcomeReason = dropReason || 'sample_rate';
     // A standalone span is sent on its own and never becomes a transaction.
-    // TODO(standalone): drop the `isStandalone` check once the static trace lifecycle is gone.
+    // TODO(v12): Drop the `isStandalone` check once the static trace lifecycle is gone.
     if (!hasSpanStreamingEnabled(client) && !spanArguments.isStandalone) {
       client.recordDroppedEvent(outcomeReason, 'transaction');
     }