Skip to content

Commit 028db6f

Browse files
rexxarss1gr1d
rexxars
and
s1gr1d
authored
ref: Fix typos in various places (#14144)
Fixed one typo and then found another, and another… Most of these aren't exactly critical as they're mostly in comments/tests, but some are in TSDocs and package descriptions which might be nice to get fixed. Co-authored-by: Sigrid Huemer <32902192+s1gr1d@users.noreply.github.com>
1 parent 7f7d609 commit 028db6f

File tree

148 files changed

+206
-206
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

148 files changed

+206
-206
lines changed

‎dev-packages/browser-integration-tests/suites/feedback/attachTo/init.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,4 @@ Sentry.init({
1414
integrations: [feedback],
1515
});
1616

17-
feedback.attachTo('#custom-feedback-buttom');
17+
feedback.attachTo('#custom-feedback-button');

‎dev-packages/browser-integration-tests/suites/feedback/attachTo/template.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,6 @@
44
<meta charset="utf-8" />
55
</head>
66
<body>
7-
<button type="button" id="custom-feedback-buttom">Show feedback!</button>
7+
<button type="button" id="custom-feedback-button">Show feedback!</button>
88
</body>
99
</html>

‎dev-packages/browser-integration-tests/suites/feedback/attachTo/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ sentryTest('should capture feedback with custom button', async ({ getLocalTestUr
2626
const url = await getLocalTestUrl({ testDir: __dirname });
2727

2828
await page.goto(url);
29-
await page.locator('#custom-feedback-buttom').click();
29+
await page.locator('#custom-feedback-button').click();
3030
await page.waitForSelector(':visible:text-is("Report a Bug")');
3131

3232
expect(await page.locator(':visible:text-is("Report a Bug")').count()).toEqual(1);

‎dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/non-string-arg/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ sentryTest(
88
'should catch onerror calls with non-string first argument gracefully',
99
async ({ getLocalTestPath, page, browserName }) => {
1010
if (browserName === 'webkit') {
11-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
11+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1212
sentryTest.skip();
1313
}
1414

‎dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/rethrown/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ sentryTest(
88
'should NOT catch an exception already caught [but rethrown] via Sentry.captureException',
99
async ({ getLocalTestPath, page, browserName }) => {
1010
if (browserName === 'webkit') {
11-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
11+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1212
sentryTest.skip();
1313
}
1414

‎dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/syntax-errors/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
66

77
sentryTest('should catch syntax errors', async ({ getLocalTestPath, page, browserName }) => {
88
if (browserName === 'webkit') {
9-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
9+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1010
sentryTest.skip();
1111
}
1212

‎dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-errors/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
66

77
sentryTest('should catch thrown errors', async ({ getLocalTestPath, page, browserName }) => {
88
if (browserName === 'webkit') {
9-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
9+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1010
sentryTest.skip();
1111
}
1212

‎dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-objects/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
66

77
sentryTest('should catch thrown objects', async ({ getLocalTestPath, page, browserName }) => {
88
if (browserName === 'webkit') {
9-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
9+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1010
sentryTest.skip();
1111
}
1212

‎dev-packages/browser-integration-tests/suites/public-api/instrumentation/onError/thrown-strings/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { getFirstSentryEnvelopeRequest, runScriptInSandbox } from '../../../../.
66

77
sentryTest('should catch thrown strings', async ({ getLocalTestPath, page, browserName }) => {
88
if (browserName === 'webkit') {
9-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
9+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1010
sentryTest.skip();
1111
}
1212

‎dev-packages/browser-integration-tests/suites/public-api/startSpan/error-sync/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ sentryTest(
1212
'should capture an error within a sync startSpan callback',
1313
async ({ getLocalTestPath, page, browserName }) => {
1414
if (browserName === 'webkit') {
15-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
15+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1616
sentryTest.skip();
1717
}
1818

‎dev-packages/browser-integration-tests/suites/replay/sessionInactive/init.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ Sentry.init({
1818

1919
window.Replay._replay.timeouts = {
2020
sessionIdlePause: 1000, // this is usually 5min, but we want to test this with shorter times
21-
sessionIdleExpire: 900000, // defayult: 15min
21+
sessionIdleExpire: 900000, // default: 15min
2222
};

‎dev-packages/browser-integration-tests/suites/replay/throttleBreadcrumbs/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ sentryTest(
3030
const spans = [...res0.performanceSpans, ...res1.performanceSpans];
3131
expect(breadcrumbs.filter(breadcrumb => breadcrumb.category === 'replay.throttled').length).toBe(1);
3232
// replay.throttled breadcrumb does *not* use the throttledAddEvent as we
33-
// alwants want that breadcrumb to be present in replay
33+
// always want that breadcrumb to be present in replay
3434
expect(breadcrumbs.length + spans.length).toBe(THROTTLE_LIMIT + 1);
3535
},
3636
);

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/error/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ sentryTest(
1111
'should put the pageload transaction name onto an error event caught during pageload',
1212
async ({ getLocalTestPath, page, browserName }) => {
1313
if (browserName === 'webkit') {
14-
// This test fails on Webkit as erros thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
14+
// This test fails on Webkit as errors thrown from `runScriptInSandbox` are Script Errors and skipped by Sentry
1515
sentryTest.skip();
1616
}
1717

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/interactions/subject.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
const blockUI = e => {
22
const startTime = Date.now();
33

4-
function getElasped() {
4+
function getElapsed() {
55
const time = Date.now();
66
return time - startTime;
77
}
88

9-
while (getElasped() < 70) {
9+
while (getElapsed() < 70) {
1010
//
1111
}
1212

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-disabled/assets/script.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
(() => {
22
const startTime = Date.now();
33

4-
function getElasped() {
4+
function getElapsed() {
55
const time = Date.now();
66
return time - startTime;
77
}
88

9-
while (getElasped() < 101) {
9+
while (getElapsed() < 101) {
1010
//
1111
}
1212
})();

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-animation-frame-non-chromium/assets/script.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
(() => {
22
const startTime = Date.now();
33

4-
function getElasped() {
4+
function getElapsed() {
55
const time = Date.now();
66
return time - startTime;
77
}
88

9-
while (getElasped() < 101) {
9+
while (getElapsed() < 101) {
1010
//
1111
}
1212
})();

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-disabled/assets/script.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
(() => {
22
const startTime = Date.now();
33

4-
function getElasped() {
4+
function getElapsed() {
55
const time = Date.now();
66
return time - startTime;
77
}
88

9-
while (getElasped() < 101) {
9+
while (getElapsed() < 101) {
1010
//
1111
}
1212
})();

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-enabled/assets/script.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
(() => {
22
const startTime = Date.now();
33

4-
function getElasped() {
4+
function getElapsed() {
55
const time = Date.now();
66
return time - startTime;
77
}
88

9-
while (getElasped() < 105) {
9+
while (getElapsed() < 105) {
1010
//
1111
}
1212
})();

‎dev-packages/browser-integration-tests/suites/tracing/browserTracingIntegration/long-tasks-no-animation-frame/assets/script.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
(() => {
22
const startTime = Date.now();
33

4-
function getElasped() {
4+
function getElapsed() {
55
const time = Date.now();
66
return time - startTime;
77
}
88

9-
while (getElasped() < 105) {
9+
while (getElapsed() < 105) {
1010
//
1111
}
1212
})();

‎dev-packages/browser-integration-tests/suites/tracing/metrics/pageload-resource-spans/test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ sentryTest('should add resource spans to pageload transaction', async ({ getLoca
2020
const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
2121
const resourceSpans = eventData.spans?.filter(({ op }) => op?.startsWith('resource'));
2222

23-
// Webkit 16.0 (which is linked to Playwright 1.27.1) consistently creates 2 consectutive spans for `css`,
23+
// Webkit 16.0 (which is linked to Playwright 1.27.1) consistently creates 2 consecutive spans for `css`,
2424
// so we need to check for 3 or 4 spans.
2525
if (browser.browserType().name() === 'webkit') {
2626
expect(resourceSpans?.length).toBeGreaterThanOrEqual(3);

‎dev-packages/browser-integration-tests/utils/helpers.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -275,8 +275,8 @@ export function shouldSkipMetricsTest(): boolean {
275275

276276
/**
277277
* Waits until a number of requests matching urlRgx at the given URL arrive.
278-
* If the timout option is configured, this function will abort waiting, even if it hasn't reveived the configured
279-
* amount of requests, and returns all the events recieved up to that point in time.
278+
* If the timeout option is configured, this function will abort waiting, even if it hasn't received the configured
279+
* amount of requests, and returns all the events received up to that point in time.
280280
*/
281281
async function getMultipleRequests<T>(
282282
page: Page,

‎dev-packages/browser-integration-tests/utils/replayEventTemplates.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ const DEFAULT_REPLAY_EVENT = {
4444
* This is useful for testing multi-segment replays to not repeat most of the properties that don't change
4545
* throughout the replay segments.
4646
*
47-
* Note: The benfit of this approach over expect.objectContaining is that,
47+
* Note: The benefit of this approach over expect.objectContaining is that,
4848
* we'll catch if properties we expect to stay the same actually change.
4949
*
5050
* @param customExpectedReplayEvent overwrite the default values with custom values (e.g. segment_id)

‎dev-packages/e2e-tests/test-applications/astro-4/src/pages/index.astro

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import Layout from '../layouts/Layout.astro';
88
<ul role="list" style="display: flex; flex-direction: column;">
99
<a href="/ssr-error">SSR Error</a>
1010
<a href="/endpoint-error">Endpoint Error</a>
11-
<a href="/client-error">Cliennt Error</a>
11+
<a href="/client-error">Client Error</a>
1212
<a href="/test-ssr">SSR page</a>
1313
<a href="/test-static" title="static page">Static Page</a>
1414
</ul>

‎dev-packages/e2e-tests/test-applications/nextjs-14/tests/generation-functions.test.ts

+6-6
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { expect, test } from '@playwright/test';
22
import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
33

4-
test('Should emit a span for a generateMetadata() function invokation', async ({ page }) => {
4+
test('Should emit a span for a generateMetadata() function invocation', async ({ page }) => {
55
const testTitle = 'should-emit-span';
66

77
const transactionPromise = waitForTransaction('nextjs-14', async transactionEvent => {
@@ -29,7 +29,7 @@ test('Should emit a span for a generateMetadata() function invokation', async ({
2929
expect(pageTitle).toBe(testTitle);
3030
});
3131

32-
test('Should send a transaction and an error event for a faulty generateMetadata() function invokation', async ({
32+
test('Should send a transaction and an error event for a faulty generateMetadata() function invocation', async ({
3333
page,
3434
}) => {
3535
const testTitle = 'should-emit-error';
@@ -59,7 +59,7 @@ test('Should send a transaction and an error event for a faulty generateMetadata
5959
expect(transactionEvent.tags?.['my-global-scope-isolated-tag']).not.toBeDefined();
6060
});
6161

62-
test('Should send a transaction event for a generateViewport() function invokation', async ({ page }) => {
62+
test('Should send a transaction event for a generateViewport() function invocation', async ({ page }) => {
6363
const testTitle = 'floob';
6464

6565
const transactionPromise = waitForTransaction('nextjs-14', async transactionEvent => {
@@ -83,7 +83,7 @@ test('Should send a transaction event for a generateViewport() function invokati
8383
);
8484
});
8585

86-
test('Should send a transaction and an error event for a faulty generateViewport() function invokation', async ({
86+
test('Should send a transaction and an error event for a faulty generateViewport() function invocation', async ({
8787
page,
8888
}) => {
8989
const testTitle = 'blargh';
@@ -109,7 +109,7 @@ test('Should send a transaction and an error event for a faulty generateViewport
109109
expect(errorEvent.transaction).toBe('Page.generateViewport (/generation-functions)');
110110
});
111111

112-
test('Should send a transaction event with correct status for a generateMetadata() function invokation with redirect()', async ({
112+
test('Should send a transaction event with correct status for a generateMetadata() function invocation with redirect()', async ({
113113
page,
114114
}) => {
115115
const testTitle = 'redirect-foobar';
@@ -126,7 +126,7 @@ test('Should send a transaction event with correct status for a generateMetadata
126126
expect((await transactionPromise).contexts?.trace?.status).toBe('ok');
127127
});
128128

129-
test('Should send a transaction event with correct status for a generateMetadata() function invokation with notfound()', async ({
129+
test('Should send a transaction event with correct status for a generateMetadata() function invocation with notfound()', async ({
130130
page,
131131
}) => {
132132
const testTitle = 'notfound-foobar';

‎dev-packages/e2e-tests/test-applications/node-otel-custom-sampler/src/instrument.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Sentry.init({
1212
debug: !!process.env.DEBUG,
1313
tunnel: `http://localhost:3031/`, // proxy server
1414
skipOpenTelemetrySetup: true,
15-
// By defining _any_ sample rate, tracing intergations will be added by default
15+
// By defining _any_ sample rate, tracing integrations will be added by default
1616
tracesSampleRate: 0,
1717
});
1818

‎dev-packages/e2e-tests/test-applications/node-otel-without-tracing/tests/errors.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ test('Sends correct error event', async ({ baseURL }) => {
2020
url: 'http://localhost:3030/test-exception/123',
2121
});
2222

23-
// This is unparametrized here because we do not have the express instrumentation
23+
// This is unparameterized here because we do not have the express instrumentation
2424
expect(errorEvent.transaction).toEqual('GET /test-exception/123');
2525

2626
expect(errorEvent.contexts?.trace).toEqual({

‎dev-packages/e2e-tests/test-applications/react-router-6/tests/sse.test.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ test('Waits for sse streaming when sse has been explicitly aborted', async ({ pa
6565
expect(resolveDuration).toBe(0);
6666
expect(resolveBodyDuration).toBe(0);
6767

68-
// validate abort eror was thrown by inspecting console
68+
// validate abort error was thrown by inspecting console
6969
const consoleBreadcrumb = rootSpan.breadcrumbs?.find(breadcrumb => breadcrumb.category === 'console');
7070
expect(consoleBreadcrumb?.message).toBe('Could not fetch sse AbortError: BodyStreamBuffer was aborted');
7171
});

‎dev-packages/node-integration-tests/suites/express/tracing/tracesSampler/server.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Sentry.init({
88
tracesSampler: samplingContext => {
99
// The name we get here is inferred at span creation time
1010
// At this point, we sadly do not have a http.route attribute yet,
11-
// so we infer the name from the unparametrized route instead
11+
// so we infer the name from the unparameterized route instead
1212
return (
1313
samplingContext.name === 'GET /test/123' &&
1414
samplingContext.attributes['sentry.op'] === 'http.server' &&

‎dev-packages/rollup-utils/plugins/extractPolyfillsPlugin.mjs

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ const POLYFILL_NAMES = new Set([
1313
]);
1414

1515
/**
16-
* Create a plugin which will replace function definitions of any of the above funcions with an `import` or `require`
16+
* Create a plugin which will replace function definitions of any of the above functions with an `import` or `require`
1717
* statement pulling them in from a central source. Mimics tsc's `importHelpers` option.
1818
*/
1919
export function makeExtractPolyfillsPlugin() {
@@ -40,7 +40,7 @@ export function makeExtractPolyfillsPlugin() {
4040
return null;
4141
}
4242

43-
// The index.js file of the tuils package will include identifiers named after polyfills so we would inject the
43+
// The index.js file of the utils package will include identifiers named after polyfills so we would inject the
4444
// polyfills, however that would override the exports so we should just skip that file.
4545
const isUtilsPackage = process.cwd().endsWith(`packages${path.sep}utils`);
4646
if (isUtilsPackage && sourceFile === 'index.js') {

‎dev-packages/size-limit-gh-action/index.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ async function run() {
130130
const sizeLimitComment = await fetchPreviousComment(octokit, repo, pr);
131131

132132
if (sizeLimitComment) {
133-
core.debug('Found existing size limit comment, udpating it instead of creating a new one...');
133+
core.debug('Found existing size limit comment, updating it instead of creating a new one...');
134134
}
135135

136136
const shouldComment =

‎docs/creating-a-new-sdk.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ Some of the following concepts may be relevant to your SDK:
5151
- **Redirects**: If possible, we want to skip redirects. This means that if a user navigates to `/`, and this redirects
5252
the user internally to `/dashboard`, we only want to capture a single `/` navigation/pageload.
5353
- **Route Params**: Routes should be parametrized, which means that instead of `/users/123` we want to capture
54-
`/users/:id` or simmilar.
54+
`/users/:id` or similar.
5555
- **Query Params**: Query params should generally be removed from the route.
5656

5757
#### Component Tracking
@@ -88,7 +88,7 @@ own instrumentation to capture `http.server` spans.
8888
Some of the following concepts may be relevant to your SDK:
8989

9090
- **Route Params**: Routes should be parametrized, which means that instead of `/users/123` we want to capture
91-
`/users/:id` or simmilar.
91+
`/users/:id` or similar.
9292
- **Query Params**: Query params should generally be removed from the route.
9393

9494
#### Middleware Tracking

‎docs/migration/feedback.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# End of Feedback Beta
22

3-
With the release of 8.0.0, Sentry Feedback is now out of Beta. This means that the usual stabilty guarantees apply.
3+
With the release of 8.0.0, Sentry Feedback is now out of Beta. This means that the usual stability guarantees apply.
44

55
Feedback 8.0.0 requires server version 24.4.2 and above.
66

@@ -89,7 +89,7 @@ function attachTo(button: HTMLElement) {
8989
onFormClose: () => {
9090
widget.close();
9191
},
92-
onFormSubmited: () => {
92+
onFormSubmitted: () => {
9393
widget.removeFromDom();
9494
}
9595
});

‎docs/v8-node.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,7 @@ const { SentrySpanProcessor, SentryPropagator, SentryContextManager, SentrySampl
136136
// We need a custom span processor
137137
provider.addSpanProcessor(new SentrySpanProcessor());
138138
// We need a custom propagator and context manager
139-
provier.register({
139+
provider.register({
140140
propagator: new SentryPropagator(),
141141
contextManager: new SentryContextManager(),
142142
});

0 commit comments

Comments
 (0)