Skip to content

Commit

Permalink
move noUnusedLocals=true from IDE to compile
Browse files Browse the repository at this point in the history
Adjusted a few files that didn't follow this rule to get it to compile.
  • Loading branch information
jkasten2 committed Sep 20, 2024
1 parent 7602f41 commit aec2837
Show file tree
Hide file tree
Showing 7 changed files with 2 additions and 10 deletions.
2 changes: 1 addition & 1 deletion build/config/tsconfig.base.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
"noImplicitAny": false,
"noImplicitReturns": true,
"noImplicitThis": true,
"noUnusedLocals": false,
"noUnusedLocals": true,
"noUnusedParameters": false,
"allowSyntheticDefaultImports": true,
"allowUnreachableCode": false,
Expand Down
4 changes: 1 addition & 3 deletions src/page/bell/Bell.ts
Original file line number Diff line number Diff line change
Expand Up @@ -494,9 +494,7 @@ export default class Bell {

const isPushEnabled =
await OneSignal.context.subscriptionManager.isPushNotificationsEnabled();
const doNotPrompt = DismissHelper.wasPromptOfTypeDismissed(
DismissPrompt.Push,
);
DismissHelper.wasPromptOfTypeDismissed(DismissPrompt.Push);

// Resize to small instead of specified size if enabled, otherwise there's a jerking motion
// where the bell, at a different size than small, jerks sideways to go from large -> small or medium -> small
Expand Down
2 changes: 0 additions & 2 deletions src/page/bell/Dialog.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
import bowser from 'bowser';

import OneSignalEvent from '../../shared/services/OneSignalEvent';
import SdkEnvironment from '../../shared/managers/SdkEnvironment';
import {
Expand Down
1 change: 0 additions & 1 deletion src/page/managers/PromptsManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import {
} from '../../shared/config/constants';
import { DismissHelper } from '../../shared/helpers/DismissHelper';
import InitHelper from '../../shared/helpers/InitHelper';
import MainHelper from '../../shared/helpers/MainHelper';
import PromptsHelper from '../../shared/helpers/PromptsHelper';
import Log from '../../shared/libraries/Log';
import {
Expand Down
1 change: 0 additions & 1 deletion src/shared/helpers/EventHelper.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import OneSignalApiShared from '../api/OneSignalApiShared';
import Log from '../libraries/Log';
import { CustomLinkManager } from '../managers/CustomLinkManager';
import { ContextSWInterface } from '../models/ContextSW';
Expand Down
1 change: 0 additions & 1 deletion src/shared/utils/utils.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import SdkEnvironment from '../managers/SdkEnvironment';
import { WindowEnvironmentKind } from '../models/WindowEnvironmentKind';
import Database from '../services/Database';
import { OneSignalUtils } from './OneSignalUtils';
import { PermissionUtils } from './PermissionUtils';
import { Utils } from '../context/Utils';
Expand Down
1 change: 0 additions & 1 deletion tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
"strictNullChecks": true,
"checkJs": true,
"noImplicitAny": true,
"noUnusedLocals": true,
"noUnusedParameters": true,
"strictPropertyInitialization": true,
"baseUrl": ".",
Expand Down

0 comments on commit aec2837

Please sign in to comment.