Skip to content

Commit

Permalink
repo: suppress 'runtime-import-check' errors
Browse files Browse the repository at this point in the history
The commit establishes a baseline for the custom `runtime-import-check`
eslint rule. The changes are mainly to suppress the errors for the time
being so they can be tracked and fixed.

Signed-off-by: vince-fugnitto <[email protected]>
  • Loading branch information
vince-fugnitto committed Sep 16, 2021
1 parent 4989293 commit 7b8b527
Show file tree
Hide file tree
Showing 15 changed files with 40 additions and 0 deletions.
3 changes: 3 additions & 0 deletions packages/core/src/common/command.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ import { injectable, inject, named } from 'inversify';
import { Event, Emitter, WaitUntilEvent } from './event';
import { Disposable, DisposableCollection } from './disposable';
import { ContributionProvider } from './contribution-provider';

// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { nls } from '../browser/nls';

/**
Expand Down
3 changes: 3 additions & 0 deletions packages/core/src/common/quick-pick-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@
* SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
********************************************************************************/
import { Event } from './event';

// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { QuickInputButtonHandle, QuickPick, QuickPickItem, QuickPickOptions } from '../browser/quick-input/quick-input-service';

export const quickPickServicePath = '/services/quickPick';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
* SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
********************************************************************************/

// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { NewWindowOptions } from '../browser/window/window-service';

export const electronMainWindowServicePath = '/services/electron-window';
Expand Down
4 changes: 4 additions & 0 deletions packages/core/src/electron-main/electron-main-application.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,11 @@ import { ContributionProvider } from '../common/contribution-provider';
import { ElectronSecurityTokenService } from './electron-security-token-service';
import { ElectronSecurityToken } from '../electron-common/electron-token';
import Storage = require('electron-store');

// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { DEFAULT_WINDOW_HASH } from '../browser/window/window-service';

const createYargs: (argv?: string[], cwd?: string) => Argv = require('yargs/yargs');

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@ import { shell } from '../../shared/electron';
import { injectable, inject } from 'inversify';
import { ElectronMainWindowService } from '../electron-common/electron-main-window-service';
import { ElectronMainApplication } from './electron-main-application';

// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { NewWindowOptions } from '../browser/window/window-service';

@injectable()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ type DialogProperties = 'openFile' | 'openDirectory' | 'multiSelections' | 'show
// and at packaging time, clients can decide whether they need the native or the browser-based
// solution.
//
// TODO: fix import.
// eslint-disable-next-line @theia/runtime-import-check
import { FileUri } from '@theia/core/lib/node/file-uri';

@injectable()
Expand Down
6 changes: 6 additions & 0 deletions packages/output/src/common/output-channel.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,15 @@ import URI from '@theia/core/lib/common/uri';
import { Deferred } from '@theia/core/lib/common/promise-util';
import { Resource, ResourceResolver } from '@theia/core/lib/common/resource';
import { Emitter, Event, Disposable, DisposableCollection } from '@theia/core';
// TODO: fix import.
// eslint-disable-next-line @theia/runtime-import-check
import { MonacoEditorModel } from '@theia/monaco/lib/browser/monaco-editor-model';
// TODO: fix import.
// eslint-disable-next-line @theia/runtime-import-check
import { MonacoTextModelService, IReference } from '@theia/monaco/lib/browser/monaco-text-model-service';
import { OutputUri } from './output-uri';
// TODO: fix import.
// eslint-disable-next-line @theia/runtime-import-check
import { OutputResource } from '../browser/output-resource';
import { OutputPreferences } from './output-preferences';

Expand Down
3 changes: 3 additions & 0 deletions packages/output/src/common/output-preferences.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,15 @@
********************************************************************************/

import { interfaces } from '@theia/core/shared/inversify';

// TODO: fix import.
import {
createPreferenceProxy,
PreferenceProxy,
PreferenceService,
PreferenceContribution,
PreferenceSchema
// eslint-disable-next-line @theia/runtime-import-check
} from '@theia/core/lib/browser/preferences';

export const OutputConfigSchema: PreferenceSchema = {
Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-dev/src/common/plugin-dev-protocol.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@
********************************************************************************/

import { JsonRpcServer } from '@theia/core/lib/common/messaging/proxy-factory';
// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { DebugPluginConfiguration } from '@theia/debug/lib/browser/debug-contribution';
import { PluginMetadata } from '@theia/plugin-ext/lib/common/plugin-protocol';

Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-dev/src/node/hosted-instance-manager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,8 @@ import { FileUri } from '@theia/core/lib/node/file-uri';
import { LogType } from '@theia/plugin-ext/lib/common/types';
import { HostedPluginSupport } from '@theia/plugin-ext/lib/hosted/node/hosted-plugin';
import { MetadataScanner } from '@theia/plugin-ext/lib/hosted/node/metadata-scanner';
// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { DebugPluginConfiguration } from '@theia/debug/lib/browser/debug-contribution';
import { HostedPluginProcess } from '@theia/plugin-ext/lib/hosted/node/hosted-plugin-process';

Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-dev/src/node/hosted-plugin-service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ import URI from '@theia/core/lib/common/uri';
import { HostedPluginReader } from './hosted-plugin-reader';
import { HostedPluginsManager } from './hosted-plugins-manager';
import { HostedPluginSupport } from '@theia/plugin-ext/lib/hosted/node/hosted-plugin';
// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { DebugPluginConfiguration } from '@theia/debug/lib/browser/debug-contribution';

@injectable()
Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-ext/src/common/plugin-api-rpc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,8 @@ import type {
TimelineProviderDescriptor
} from '@theia/timeline/lib/common/timeline-model';
import { SerializableEnvironmentVariableCollection } from '@theia/terminal/lib/common/base-terminal-protocol';
// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { ThemeType } from '@theia/core/lib/browser/theming';
import { Disposable } from '@theia/core/lib/common/disposable';
import { PickOptions, QuickInputButtonHandle, QuickPickItem } from '@theia/core/lib/browser';
Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-ext/src/common/plugin-protocol.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,8 @@ import { IJSONSchema, IJSONSchemaSnippet } from '@theia/core/lib/common/json-sch
import { RecursivePartial } from '@theia/core/lib/common/types';
import { PreferenceSchema, PreferenceSchemaProperties } from '@theia/core/lib/common/preferences/preference-schema';
import { ProblemMatcherContribution, ProblemPatternContribution, TaskDefinition } from '@theia/task/lib/common';
// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { ColorDefinition } from '@theia/core/lib/browser/color-registry';
import { ResourceLabelFormatter } from '@theia/core/lib/common/label-protocol';

Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-ext/src/hosted/browser/worker/debug-stub.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
* SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0
********************************************************************************/

// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { DebugExtImpl } from '../../../plugin/node/debug/debug';
import { RPCProtocol } from '../../../common/rpc-protocol';

Expand Down
2 changes: 2 additions & 0 deletions packages/plugin-ext/src/hosted/node/scanners/scanner-theia.ts
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,8 @@ import {
ProblemPatternContribution,
TaskDefinition
} from '@theia/task/lib/common/task-protocol';
// TODO: fix the import.
// eslint-disable-next-line @theia/runtime-import-check
import { ColorDefinition } from '@theia/core/lib/browser/color-registry';
import { ResourceLabelFormatter } from '@theia/core/lib/common/label-protocol';
import { PluginUriFactory } from './plugin-uri-factory';
Expand Down

0 comments on commit 7b8b527

Please sign in to comment.