Skip to content

Commit 9eb0cff

Browse files
authored
ref(✂️): update knip and address new findings (#104151)
knip got better and finds more stuff now 🙌
1 parent 8794e2d commit 9eb0cff

File tree

8 files changed

+230
-438
lines changed

8 files changed

+230
-438
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,7 @@
223223
"jest-environment-jsdom": "30.0.4",
224224
"jest-fail-on-console": "3.3.1",
225225
"jest-junit": "16.0.0",
226-
"knip": "5.64.0",
226+
"knip": "5.71.0",
227227
"postcss-styled-syntax": "0.7.0",
228228
"react-refresh": "0.18.0",
229229
"stylelint": "16.10.0",

pnpm-lock.yaml

Lines changed: 224 additions & 221 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

static/app/actionCreators/modal.tsx

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ import type {ReprocessEventModalOptions} from 'sentry/components/modals/reproces
1212
import type {TokenRegenerationConfirmationModalProps} from 'sentry/components/modals/tokenRegenerationConfirmationModal';
1313
import type {AddToDashboardModalProps} from 'sentry/components/modals/widgetBuilder/addToDashboardModal';
1414
import type {LinkToDashboardModalProps} from 'sentry/components/modals/widgetBuilder/linkToDashboardModal';
15-
import type {OverwriteWidgetModalProps} from 'sentry/components/modals/widgetBuilder/overwriteWidgetModal';
1615
import type {WidgetViewerModalOptions} from 'sentry/components/modals/widgetViewerModal';
1716
import type {ConsoleModalProps} from 'sentry/components/onboarding/consoleModal';
1817
import type {Category} from 'sentry/components/platformPicker';
@@ -263,19 +262,6 @@ export async function openInviteMissingMembersModal({
263262
});
264263
}
265264

266-
export async function openWidgetBuilderOverwriteModal(
267-
options: OverwriteWidgetModalProps
268-
) {
269-
const {default: Modal, modalCss} = await import(
270-
'sentry/components/modals/widgetBuilder/overwriteWidgetModal'
271-
);
272-
273-
openModal(deps => <Modal {...deps} {...options} />, {
274-
closeEvents: 'escape-key',
275-
modalCss,
276-
});
277-
}
278-
279265
export async function openAddToDashboardModal(options: AddToDashboardModalProps) {
280266
const {default: Modal, modalCss} = await import(
281267
'sentry/components/modals/widgetBuilder/addToDashboardModal'
@@ -328,7 +314,7 @@ export async function demoSignupModal(options: ModalOptions = {}) {
328314
openModal(deps => <Modal {...deps} {...options} />, {modalCss});
329315
}
330316

331-
export type DemoEndModalOptions = {
317+
type DemoEndModalOptions = {
332318
tour: string;
333319
};
334320

@@ -389,7 +375,7 @@ export async function openCreateReleaseIntegration(
389375
openModal(deps => <Modal {...deps} {...options} />);
390376
}
391377

392-
export type NavigateToExternalLinkModalOptions = {
378+
type NavigateToExternalLinkModalOptions = {
393379
linkText: string;
394380
};
395381

@@ -488,7 +474,7 @@ export async function openPrivateGamingSdkAccessModal(
488474
openModal(deps => <PrivateGamingSdkAccessModal {...deps} {...options} />);
489475
}
490476

491-
export type InsightInfoModalOptions = {
477+
type InsightInfoModalOptions = {
492478
children: React.ReactNode;
493479
title: string;
494480
};

static/app/components/modals/widgetBuilder/overwriteWidgetModal.spec.tsx

Lines changed: 0 additions & 65 deletions
This file was deleted.

static/app/components/modals/widgetBuilder/overwriteWidgetModal.tsx

Lines changed: 0 additions & 72 deletions
This file was deleted.

static/app/utils/issueTypeConfig/index.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ const issueTypeConfig: Config = {
112112
* errors that may otherwise be difficult to debug. For example, common framework
113113
* errors that have no stack trace.
114114
*/
115-
export function shouldShowCustomErrorResourceConfig(
115+
function shouldShowCustomErrorResourceConfig(
116116
params: GetConfigForIssueTypeParams,
117117
project: Project
118118
): boolean {

static/app/views/dashboards/widgetBuilder/widgetLibrary/card.tsx

Lines changed: 0 additions & 60 deletions
This file was deleted.

static/app/views/dashboards/widgetLibrary/data.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import type {Widget} from 'sentry/views/dashboards/types';
66
import {DisplayType, WidgetType} from 'sentry/views/dashboards/types';
77
import {hasDatasetSelector} from 'sentry/views/dashboards/utils';
88

9-
export type WidgetTemplate = Widget & {
9+
type WidgetTemplate = Widget & {
1010
description: string;
1111
};
1212

0 commit comments

Comments
 (0)