Fix for a crash when loading environments with no info (#18598) · randomir/vscode-python@a4c3170 · GitHub

randomir / vscode-python Public

forked from dwavesystems/vscode-python

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit a4c3170

Browse filesBrowse the repository at this point in the historyBrowse files

karthiknadig

authored

Fix for a crash when loading environments with no info (microsoft#18598)

1 parent bc4880a commit a4c3170
Copy full SHA for a4c3170

2 files changed

+4-3Lines changed: 4 additions & 3 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• news/2 Fixes
  • 18594.md
• src/client/interpreter/display
  • index.ts

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎news/2 Fixes/18594.md‎

Copy file name to clipboard

+1Lines changed: 1 addition & 0 deletions

• Display the source diff
• Display the rich diff

Original file line number	Diff line number	Diff line change
@@ -0,0 +1 @@
	1	+ Fix for a crash when loading environments with no info.

Collapse file

‎src/client/interpreter/display/index.ts‎

Copy file name to clipboardExpand all lines: src/client/interpreter/display/index.ts

+3-3Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
@@ -119,11 +119,11 @@ export class InterpreterDisplay implements IInterpreterDisplay, IExtensionSingle
119	119	);
120	120	this.interpreterPath = interpreter.path;
121	121	}
122		- let text = interpreter.detailedDisplayName!;
	122	+ let text = interpreter.detailedDisplayName;
123	123	if (this.experiments.inExperimentSync(InterpreterStatusBarPosition.Pinned)) {
124		- text = text.startsWith('Python') ? text.substring('Python'.length).trim() : text;
	124	+ text = text?.startsWith('Python') ? text?.substring('Python'.length)?.trim() : text;
125	125	}
126		- this.statusBar.text = text;
	126	+ this.statusBar.text = text!;
127	127	this.currentlySelectedInterpreterPath = interpreter.path;
128	128	} else {
129	129	this.statusBar.tooltip = '';

0 commit comments

Comments

0 (0)