mirror of
https://github.com/johannesjo/super-productivity.git
synced 2026-01-23 02:36:05 +00:00
chore(lint): fix eslint errors across codebase
- Update eslint-disable directives from no-var-requires to no-require-imports - Add naming-convention disable for test files with date/hyphenated keys - Remove references to non-existent local-rules
This commit is contained in:
parent
09c992f2cb
commit
004c10586b
7 changed files with 5 additions and 6 deletions
|
|
@ -80,7 +80,6 @@ export class GlobalConfigEffects {
|
|||
// Intentional: UI config should apply from any source, dispatch: false
|
||||
applyLanguageFromState$ = createEffect(
|
||||
() =>
|
||||
// eslint-disable-next-line local-rules/require-hydration-guard
|
||||
this._store.select(selectLocalizationConfig).pipe(
|
||||
map((config) => config.lng),
|
||||
distinctUntilChanged(),
|
||||
|
|
|
|||
|
|
@ -1,3 +1,4 @@
|
|||
/* eslint-disable @typescript-eslint/naming-convention */
|
||||
import { Task, TaskState } from '../task.model';
|
||||
import { initialTaskState, taskReducer } from './task.reducer';
|
||||
import * as fromActions from './task.actions';
|
||||
|
|
|
|||
|
|
@ -33,7 +33,7 @@ export class UserProfileService {
|
|||
private _syncWrapperServiceCache: SyncWrapperService | null = null;
|
||||
private get _syncWrapperService(): SyncWrapperService {
|
||||
if (!this._syncWrapperServiceCache) {
|
||||
// eslint-disable-next-line @typescript-eslint/no-var-requires
|
||||
// eslint-disable-next-line @typescript-eslint/no-require-imports
|
||||
const { SyncWrapperService: SWS } = require('../../imex/sync/sync-wrapper.service');
|
||||
this._syncWrapperServiceCache = this._injector.get(SWS);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -513,7 +513,6 @@ describe('LWW Update Store Application Integration', () => {
|
|||
type: '[UNKNOWN_TYPE] LWW Update',
|
||||
id: 'unknown1',
|
||||
title: 'Unknown',
|
||||
// eslint-disable-next-line local-rules/require-entity-registry -- intentionally testing unknown entity type handling
|
||||
meta: { isPersistent: true, entityType: 'UNKNOWN_TYPE', entityId: 'unknown1' },
|
||||
};
|
||||
|
||||
|
|
|
|||
|
|
@ -12,7 +12,7 @@ describe('isRelatedModelDataValid', () => {
|
|||
spyOn(PFLog, 'warn');
|
||||
spyOn(PFLog, 'err');
|
||||
spyOn(PFLog, 'critical');
|
||||
/* eslint-disable @typescript-eslint/no-var-requires */
|
||||
/* eslint-disable @typescript-eslint/no-require-imports */
|
||||
// Reset modules to allow re-importing with mocks
|
||||
// @ts-ignore
|
||||
delete require.cache[require.resolve('../../util/dev-error')];
|
||||
|
|
@ -33,7 +33,7 @@ describe('isRelatedModelDataValid', () => {
|
|||
// @ts-ignore
|
||||
isRelatedModelDataValid =
|
||||
require('./is-related-model-data-valid').isRelatedModelDataValid;
|
||||
/* eslint-enable @typescript-eslint/no-var-requires */
|
||||
/* eslint-enable @typescript-eslint/no-require-imports */
|
||||
});
|
||||
|
||||
it('should handle null data gracefully', () => {
|
||||
|
|
|
|||
|
|
@ -567,6 +567,7 @@ describe('PluginBridgeService', () => {
|
|||
});
|
||||
*/
|
||||
|
||||
/* eslint-disable @typescript-eslint/naming-convention */
|
||||
// Active tests for setCounter fix (issue #5812)
|
||||
import { TestBed } from '@angular/core/testing';
|
||||
import { MockStore, provideMockStore } from '@ngrx/store/testing';
|
||||
|
|
|
|||
|
|
@ -309,7 +309,6 @@ describe('lwwUpdateMetaReducer', () => {
|
|||
type: '[UNKNOWN_ENTITY] LWW Update',
|
||||
id: 'unknown-1',
|
||||
title: 'Updated Title',
|
||||
// eslint-disable-next-line local-rules/require-entity-registry -- intentionally testing unknown entity type handling
|
||||
meta: { isPersistent: true, entityType: 'UNKNOWN_ENTITY', entityId: 'unknown-1' },
|
||||
};
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue