second pass of --fix

Author: Sue Burt

src/server/api/bannerRouter.ts

@@ -19,12 +19,12 @@ import { selectAmountsTestVariant } from '../lib/ab';
 import { getDeviceType } from '../lib/deviceType';
 import { baseUrl } from '../lib/env';
 import type { TickerDataProvider } from '../lib/fetchTickerData';
+import { getArticleViewCounts } from '../lib/history';
 import type { Params } from '../lib/params';
 import { getQueryParams } from '../lib/params';
 import { buildBannerCampaignCode } from '../lib/tracking';
 import type { BannerDeployTimesProvider } from '../tests/banners/bannerDeployTimes';
 import { selectBannerTest } from '../tests/banners/bannerSelection';
-import { getArticleViewCounts } from '../lib/history';
 import { getDesignForVariant } from '../tests/banners/channelBannerTests';
 import type { Debug } from '../tests/epics/epicSelection';
 import type { ValueProvider } from '../utils/valueReloader';

src/server/middleware/errorHandling.ts

@@ -7,7 +7,7 @@ export const errorHandling = (
     res: express.Response,
     // Error handling middleware in Express needs to take 4 arguments in the handler
     // for it to run when `next()` function is called in the route handler
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+     
     next: express.NextFunction,
 ): void => {
     const message = error.message || error.toString();

src/shared/types/props/epic.ts

@@ -21,7 +21,6 @@ export interface EpicProps {
     tracking: Tracking;
     countryCode?: string;
     articleCounts: ArticleCounts;
-    // eslint-disable-next-line @typescript-eslint/ban-types
     onReminderOpen?: Function;
     fetchEmail?: () => Promise<string | null>;
     submitComponentEvent?: (componentEvent: OphanComponentEvent) => void;