Skip to content

Commit 9fa24f9

Browse files
chore: rename java.dependency.enableDependencyDiagnostics
... to java.dependency.enableDependencyCheckup` as `Diagnostics` is used as a VSCode-specific concept
1 parent 5e0f4b0 commit 9fa24f9

File tree

5 files changed

+15
-15
lines changed

5 files changed

+15
-15
lines changed

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -333,9 +333,9 @@
333333
"description": "%configuration.java.dependency.packagePresentation%",
334334
"default": "flat"
335335
},
336-
"java.dependency.enableDependencyDiagnostics": {
336+
"java.dependency.enableDependencyCheckup": {
337337
"type": "boolean",
338-
"description": "%configuration.java.dependency.enableDependencyDiagnostics%",
338+
"description": "%configuration.java.dependency.enableDependencyCheckup%",
339339
"default": true
340340
},
341341
"java.project.exportJar.targetPath": {

package.nls.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
"configuration.java.dependency.autoRefresh": "Synchronize Java Projects explorer with changes",
4646
"configuration.java.dependency.refreshDelay": "The delay time (ms) the auto refresh is invoked when changes are detected",
4747
"configuration.java.dependency.packagePresentation": "Package presentation mode: flat or hierarchical",
48-
"configuration.java.dependency.enableDependencyDiagnostics": "Show reminders when your Java runtimes or dependencies need an upgrade.",
48+
"configuration.java.dependency.enableDependencyCheckup": "Show reminders when your Java runtimes or dependencies need an upgrade.",
4949
"configuration.java.project.explorer.showNonJavaResources": "When enabled, the explorer shows non-Java resources.",
5050
"configuration.java.project.exportJar.targetPath.customization": "The output path of the exported jar. Leave it empty if you want to manually pick the output location.",
5151
"configuration.java.project.exportJar.targetPath.workspaceFolder": "Export the jar file into the workspace folder. Its name is the same as the folder's.",

src/settings.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ export class Settings {
5858
workspace.getConfiguration("java.dependency").update("packagePresentation", PackagePresentation.Hierarchical, false);
5959
}
6060

61-
public static disableWorkspaceDependencyDiagnostics() {
62-
return workspace.getConfiguration("java.dependency").update("enableDependencyDiagnostics", false, ConfigurationTarget.Workspace);
61+
public static disableWorkspaceDependencyCheckup() {
62+
return workspace.getConfiguration("java.dependency").update("enableDependencyCheckup", false, ConfigurationTarget.Workspace);
6363
}
6464

6565
public static switchNonJavaResourceFilter(enabled: boolean): void {
@@ -112,8 +112,8 @@ export class Settings {
112112
return workspace.getConfiguration("java.dependency").get("refreshDelay", 2000);
113113
}
114114

115-
public static getEnableDependencyDiagnostics() {
116-
return workspace.getConfiguration("java.dependency").get("enableDependencyDiagnostics", true);
115+
public static getEnableDependencyCheckup() {
116+
return workspace.getConfiguration("java.dependency").get("enableDependencyCheckup", true);
117117
}
118118

119119
public static getExportJarTargetPath(): string {

src/upgrade/display/notificationManager.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ class NotificationManager implements IUpgradeIssuesRenderer {
6767
break;
6868
}
6969
case BUTTON_TEXT_DONT_SHOW_AGAIN: {
70-
Settings.disableWorkspaceDependencyDiagnostics();
70+
Settings.disableWorkspaceDependencyCheckup();
7171
break;
7272
}
7373
}
@@ -76,7 +76,7 @@ class NotificationManager implements IUpgradeIssuesRenderer {
7676
}
7777

7878
private shouldShow() {
79-
return Settings.getEnableDependencyDiagnostics()
79+
return Settings.getEnableDependencyCheckup()
8080
&& ((this.getSessionCount() ?? 0) >= 0);
8181
}
8282

src/upgrade/upgradeManager.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ import assessmentManager from "./assessmentManager";
1515
const DEFAULT_UPGRADE_PROMPT = "Upgrade Java project dependency to latest version.";
1616

1717

18-
function shouldCheckUpgrade() {
19-
return Settings.getEnableDependencyDiagnostics()
18+
function shouldRunCheckup() {
19+
return Settings.getEnableDependencyCheckup()
2020
&& !!extensions.getExtension(ExtensionName.APP_MODERNIZATION_UPGRADE_FOR_JAVA);
2121
}
2222

@@ -39,17 +39,17 @@ class UpgradeManager {
3939
}
4040

4141
public static scan() {
42-
if (!shouldCheckUpgrade()) {
42+
if (!shouldRunCheckup()) {
4343
return;
4444
}
4545
workspace.workspaceFolders?.forEach((folder) =>
46-
UpgradeManager.checkUpgradableComponents(folder)
46+
UpgradeManager.runDependencyCheckup(folder)
4747
);
4848
}
4949

50-
private static async checkUpgradableComponents(folder: WorkspaceFolder) {
50+
private static async runDependencyCheckup(folder: WorkspaceFolder) {
5151
return (instrumentOperation("java.dependency.diagnoseDependencyIssues",
52-
async () => {
52+
async (_operationId: string) => {
5353
if (!await languageServerApiManager.ready()) {
5454
return;
5555
}

0 commit comments

Comments
 (0)