Skip to content

Commit d1664d9

Browse files
AbhiPrasadlizokm
andcommitted
Apply suggestions from code review
Co-authored-by: Liza Mock <[email protected]>
1 parent 95ee57e commit d1664d9

File tree

6 files changed

+6
-6
lines changed

6 files changed

+6
-6
lines changed

src/platform-includes/performance/configure-sample-rate/javascript.mdx

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Sentry.init({
88

99
// This enables automatic instrumentation (highly recommended), but is not
1010
// necessary for purely manual usage.
11-
// If you only want manual usage remove the BrowserTracing integration and add
11+
// If you only want to use manually, remove the BrowserTracing integration and add
1212
// Sentry.addTracingExtensions() above your Sentry.init() call.
1313
integrations: [new Sentry.BrowserTracing()],
1414

src/platform-includes/performance/configure-sample-rate/javascript.react.mdx

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ Sentry.init({
66

77
// This enables automatic instrumentation (highly recommended), but is not
88
// necessary for purely manual usage.
9-
// If you only want manual usage remove the BrowserTracing integration and add
9+
// If you only want to use manually, remove the BrowserTracing integration and add
1010
// Sentry.addTracingExtensions() above your Sentry.init() call.
1111
integrations: [new Sentry.BrowserTracing()],
1212

src/platform-includes/performance/configure-sample-rate/javascript.vue.mdx

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ Sentry.init({
99

1010
// This enables automatic instrumentation (highly recommended), but is not
1111
// necessary for purely manual usage
12-
// If you only want manual usage remove the BrowserTracing integration and add
12+
// If you only want to use manually, remove the BrowserTracing integration and add
1313
// Sentry.addTracingExtensions() above your Sentry.init() call.
1414
integrations: [new Sentry.BrowserTracing()],
1515

src/platforms/javascript/common/configuration/tree-shaking/index.mdx

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ To make optional code eligible for tree shaking, you can replace various flags i
2424
`__SENTRY_TRACING__`
2525

2626
: Replacing this flag with `false` will tree shake all code in the SDK that is related to performance monitoring.
27-
**Attention:** `__SENTRY_TRACING__` must not be replaced with `false` when when you're using any performance monitoring related SDK features (e.g. `Sentry.startTransaction()`). This flag is intended to be used in combination with packages like `@sentry/next` or `@sentry/sveltekit` that automatically include performance monitoring functionality.
27+
**Attention:** `__SENTRY_TRACING__` must not be replaced with `false` when you're using any performance monitoring-related SDK features (e.g. `Sentry.startTransaction()`). This flag is intended to be used in combination with packages like `@sentry/next` or `@sentry/sveltekit`, which automatically include performance monitoring functionality.
2828

2929
<PlatformSection notSupported={["javascript.nextjs"]}>
3030

src/platforms/javascript/common/configuration/webworkers/index.mdx

+1-1
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ self.onmessage = message => {
8585

8686
### Integrations
8787

88-
Using non-default integrations inside Web Workers may not work as expected. Non-default integrations enabled outside workers' scope are not affected by worker-level configurations and will still work as expected.
88+
Note, that if you use non-default integrations inside web workers, they may not function as expected. But non-default integrations that are enabled outside of a worker’s scope won’t be affected and will function as expected.
8989

9090
### Source Maps
9191

src/platforms/javascript/guides/ember/configuration/ember-options.mdx

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ The following documentation is for Ember specific configuration, for Sentry opti
2727

2828
### Performance Monitoring Considerations
2929

30-
The Sentry tracing integration is already set up via the Ember addon with custom Ember instrumentation for routing, components, and the runloop. It sideloads Sentry performance monitoring as a chunk to instrument your application. If you would like to disable this automatic instrumentation and no longer receive the associated transactions, you can set `disablePerformance` in your config as in this example:
30+
The Sentry tracing integration is already set up via the Ember add on with custom Ember instrumentation for routing, components, and the runloop. It sideloads Sentry performance monitoring as a chunk to instrument your application. If you'd like to disable this automatic instrumentation and stop receiving the associated transactions, set `disablePerformance` in your config. See example below:
3131

3232
```javascript
3333
ENV["@sentry/ember"] = {

0 commit comments

Comments
 (0)