Skip to content

Commit 913d9f7

Browse files
committed
Add support for Diagnostic log level
Set Start-EditorServices to log only for Diagnostic log level. Add string enum support for log level so in UserSettings you get a list.
1 parent 46cbf51 commit 913d9f7

File tree

3 files changed

+17
-4
lines changed

3 files changed

+17
-4
lines changed

package.json

+8-1
Original file line numberDiff line numberDiff line change
@@ -502,8 +502,15 @@
502502
},
503503
"powershell.developer.editorServicesLogLevel": {
504504
"type": "string",
505+
"enum": [
506+
"Diagnostic",
507+
"Verbose",
508+
"Normal",
509+
"Warning",
510+
"Error"
511+
],
505512
"default": "Normal",
506-
"description": "Sets the logging verbosity level for the PowerShell Editor Services host executable. Valid values are 'Verbose', 'Normal', 'Warning', and 'Error'"
513+
"description": "Sets the logging verbosity level for the PowerShell Editor Services host executable. Valid values are 'Diagnostic', 'Verbose', 'Normal', 'Warning', and 'Error'"
507514
},
508515
"powershell.developer.editorServicesWaitForDebugger": {
509516
"type": "boolean",

scripts/Start-EditorServices.ps1

+2-2
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ param(
4444
[ValidateNotNullOrEmpty()]
4545
$LogPath,
4646

47-
[ValidateSet("Normal", "Verbose", "Error", "Diagnostic")]
47+
[ValidateSet("Diagnostic", "Normal", "Verbose", "Error", "Diagnostic")]
4848
$LogLevel,
4949

5050
[Parameter(Mandatory=$true)]
@@ -74,7 +74,7 @@ param(
7474
$minPortNumber = 10000
7575
$maxPortNumber = 30000
7676

77-
if ($LogLevel -ne "Normal") {
77+
if ($LogLevel -eq "Diagnostic") {
7878
$VerbosePreference = 'Continue'
7979
Start-Transcript (Join-Path (Split-Path $LogPath -Parent) Start-EditorServices.log) -Force
8080
}

src/logging.ts

+7-1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import vscode = require("vscode");
99
import utils = require("./utils");
1010

1111
export enum LogLevel {
12+
Diagnostic,
1213
Verbose,
1314
Normal,
1415
Warning,
@@ -65,6 +66,10 @@ export class Logger {
6566
this.writeAtLevel(LogLevel.Normal, message, ...additionalMessages);
6667
}
6768

69+
public writeDiagnostic(message: string, ...additionalMessages: string[]) {
70+
this.writeAtLevel(LogLevel.Diagnostic, message, ...additionalMessages);
71+
}
72+
6873
public writeVerbose(message: string, ...additionalMessages: string[]) {
6974
this.writeAtLevel(LogLevel.Verbose, message, ...additionalMessages);
7075
}
@@ -112,8 +117,9 @@ export class Logger {
112117

113118
private logLevelNameToValue(logLevelName: string): LogLevel {
114119
switch (logLevelName.toLowerCase()) {
115-
case "normal": return LogLevel.Normal;
120+
case "diagnostic": return LogLevel.Diagnostic;
116121
case "verbose": return LogLevel.Verbose;
122+
case "normal": return LogLevel.Normal;
117123
case "warning": return LogLevel.Warning;
118124
case "error": return LogLevel.Error;
119125
default: return LogLevel.Normal;

0 commit comments

Comments
 (0)