From 26b2a6cd2cce13a1396dbef89b1ea57e4ede97ff Mon Sep 17 00:00:00 2001 From: Alvaro Sanchez-Leon Date: Fri, 12 May 2023 12:14:58 -0400 Subject: [PATCH] Addressing review comments (offline) Signed-off-by: Alvaro Sanchez-Leon --- CHANGELOG.md | 2 +- packages/debug/src/common/debug-service.ts | 4 ++-- packages/debug/src/node/debug-service-impl.ts | 2 +- .../plugin-ext/src/main/browser/debug/plugin-debug-service.ts | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 72388e58278da..4a3e48df83c6c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,7 +5,7 @@ - [Previous Changelogs](https://github.com/eclipse-theia/theia/tree/master/doc/changelogs/) -## v1.38.0 - +## v1.38.0 - 05/25/2023 [Breaking Changes:](#breaking_changes_1.38.0) diff --git a/packages/debug/src/common/debug-service.ts b/packages/debug/src/common/debug-service.ts index 1dd6906c4deb8..52077893316ac 100644 --- a/packages/debug/src/common/debug-service.ts +++ b/packages/debug/src/common/debug-service.ts @@ -80,12 +80,12 @@ export interface DebugService extends Disposable { /** * @returns A Record of debug configuration provider types and a corresponding dynamic debug configurations array */ - provideDynamicDebugConfigurations?(folder: string | undefined): Promise>; + provideDynamicDebugConfigurations?(folder?: string): Promise>; /** * Provides a dynamic debug configuration matching the name and the provider debug type */ - fetchDynamicDebugConfiguration(name: string, type: string, folder: string | undefined): Promise; + fetchDynamicDebugConfiguration(name: string, type: string, folder?: string): Promise; /** * Resolves a [debug configuration](#DebugConfiguration) by filling in missing values diff --git a/packages/debug/src/node/debug-service-impl.ts b/packages/debug/src/node/debug-service-impl.ts index 159b504506716..f5e1e6aa1e054 100644 --- a/packages/debug/src/node/debug-service-impl.ts +++ b/packages/debug/src/node/debug-service-impl.ts @@ -71,7 +71,7 @@ export class DebugServiceImpl implements DebugService { // TODO: Support dynamic debug configurations through Theia extensions? return {}; } - fetchDynamicDebugConfiguration(name: string, type: string, folder: string | undefined): Promise { + fetchDynamicDebugConfiguration(name: string, type: string, folder?: string): Promise { // TODO: Support dynamic debug configurations through Theia extensions? return Promise.resolve(undefined); } diff --git a/packages/plugin-ext/src/main/browser/debug/plugin-debug-service.ts b/packages/plugin-ext/src/main/browser/debug/plugin-debug-service.ts index ab981e41b7425..ecb575c109200 100644 --- a/packages/plugin-ext/src/main/browser/debug/plugin-debug-service.ts +++ b/packages/plugin-ext/src/main/browser/debug/plugin-debug-service.ts @@ -142,7 +142,7 @@ export class PluginDebugService implements DebugService { return results; } - async fetchDynamicDebugConfiguration(name: string, providerType: string, folder: string | undefined): Promise { + async fetchDynamicDebugConfiguration(name: string, providerType: string, folder?: string): Promise { const pluginProviders = Array.from(this.configurationProviders.values()).filter(p => ( p.triggerKind === DebugConfigurationProviderTriggerKind.Dynamic && @@ -160,7 +160,7 @@ export class PluginDebugService implements DebugService { } } - async provideDynamicDebugConfigurations(folder: string | undefined): Promise> { + async provideDynamicDebugConfigurations(folder?: string): Promise> { const pluginProviders = Array.from(this.configurationProviders.values()).filter(p => ( p.triggerKind === DebugConfigurationProviderTriggerKind.Dynamic &&