ref: Remove @typescript-eslint/no-empty-function rule

This is IMHO not super helpful, we constantly disable it or add a noop comment for no gain.

Author: mydea

packages/cloudflare/src/opentelemetry/tracer.ts

@@ -71,8 +71,7 @@ class SentryCloudflareTracer implements Tracer {
           ? context
           : typeof fn === 'function'
             ? fn
-            : // eslint-disable-next-line @typescript-eslint/no-empty-function
-              () => {}
+            : () => {}
     ) as F;
 
     // In OTEL the semantic matches `startSpanManual` because spans are not auto-ended

packages/core/src/tracing/sentryNonRecordingSpan.ts

@@ -32,7 +32,6 @@ export class SentryNonRecordingSpan implements Span {
   }
 
   /** @inheritdoc */
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
   public end(_timestamp?: SpanTimeInput): void {}
 
   /** @inheritdoc */

packages/core/src/utils/handleCallbackErrors.ts

@@ -17,7 +17,6 @@ export function handleCallbackErrors<
 >(
   fn: Fn,
   onError: (error: unknown) => void,
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
   onFinally: () => void = () => {},
 ): ReturnType<Fn> {
   let maybePromiseResult: ReturnType<Fn>;

packages/eslint-config-sdk/src/base.js

@@ -178,7 +178,6 @@ module.exports = {
         '@typescript-eslint/explicit-member-accessibility': 'off',
         '@typescript-eslint/no-explicit-any': 'off',
         '@typescript-eslint/no-non-null-assertion': 'off',
-        '@typescript-eslint/no-empty-function': 'off',
         '@typescript-eslint/no-floating-promises': 'off',
         '@sentry-internal/sdk/no-focused-tests': 'error',
         '@sentry-internal/sdk/no-skipped-tests': 'error',

packages/feedback/src/modal/integration.tsx

@@ -22,7 +22,6 @@ function getUser(): User | undefined {
 export const feedbackModalIntegration = ((): FeedbackModalIntegration => {
   return {
     name: 'FeedbackModal',
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     setupOnce() {},
     createDialog: ({ options, screenshotIntegration, sendFeedback, shadow }) => {
       const shadowRoot = shadow as unknown as ShadowRoot;

packages/feedback/src/screenshot/integration.ts

@@ -7,7 +7,6 @@ import { ScreenshotEditorFactory } from './components/ScreenshotEditor';
 export const feedbackScreenshotIntegration = ((): FeedbackScreenshotIntegration => {
   return {
     name: 'FeedbackScreenshot',
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     setupOnce() {},
     createInput: ({ h, hooks, dialog, options }) => {
       const outputBuffer = DOCUMENT.createElement('canvas');

packages/sveltekit/src/vite/svelteConfig.ts

@@ -86,13 +86,10 @@ async function getNodeAdapterOutputDir(svelteConfig: Config): Promise<string> {
     },
     // @ts-expect-error - No need to implement the other methods
     log: {
-      // eslint-disable-next-line @typescript-eslint/no-empty-function -- this should be a noop
       minor() {},
     },
     getBuildDirectory: () => '',
-    // eslint-disable-next-line @typescript-eslint/no-empty-function -- this should be a noop
     rimraf: () => {},
-    // eslint-disable-next-line @typescript-eslint/no-empty-function -- this should be a noop
     mkdirp: () => {},
 
     config: {