aeschli
commented
1 week ago colorRegistry.releaseTest.ts has a import '../../../../workbench.desktop.main.js'; to make sure all parts of VS Code are loaded and registerColor calls are made.
I don't think we have a layer for that, but I can move the test one up to src/vs/workbench/contrib/themes/test/
Is that ok and beneficial?
There should exist no reason to disable the
local/code-import-patterns. You either need to move things into the right layer or update the configuration of the rule[ ] src/vs/base/parts/request/test/browser/request.test.ts @chrmarti
6,1: // eslint-disable-next-line local/code-import-patterns 8,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/editor/standalone/browser/standaloneTreeSitterService.ts @alexr00 6,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/platform/terminal/common/capabilities/commandDetection/promptInputModel.ts @Tyriar 13,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/platform/terminal/common/capabilities/commandDetection/terminalCommand.ts @Tyriar 10,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/platform/terminal/common/capabilities/bufferMarkCapability.ts @Tyriar 10,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/platform/terminal/common/capabilities/commandDetectionCapability.ts @Tyriar 17,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/platform/terminal/common/capabilities/partialCommandDetectionCapability.ts @Tyriar 10,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/platform/terminal/common/xterm/shellIntegrationAddon.ts @Tyriar 15,1: // eslint-disable-next-line local/code-import-patterns 20,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/codeEditor/browser/inspectEditorTokens/inspectEditorTokens.ts @alexr00 35,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/terminal/browser/xterm/markNavigationAddon.ts @Tyriar 19,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/terminal/browser/baseTerminalBackend.ts @Tyriar 17,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/terminal/browser/terminal.contribution.ts @Tyriar 59,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/workbench/contrib/terminal/browser/terminalContribExports.ts @Tyriar 8,1: // eslint-disable-next-line local/code-import-patterns 10,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/terminal/browser/terminalInstance.ts @Tyriar 95,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/terminal/browser/terminalProcessManager.ts @Tyriar 46,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/workbench/contrib/terminalContrib/quickFix/browser/quickFixAddon.ts @Tyriar 7,1: // eslint-disable-next-line local/code-import-patterns
[ ] src/vs/workbench/contrib/themes/test/node/colorRegistry.releaseTest.ts @aeschli 16,1: // eslint-disable-next-line local/code-import-patterns
[x] src/vs/workbench/services/treeSitter/browser/treeSitterTokenizationFeature.ts @alexr00 6,1: // eslint-disable-next-line local/code-import-patterns