Skip to content

Commit 5a4bf5b

Browse files
committed
remove completed comments
1 parent 7323bde commit 5a4bf5b

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/client/common/process/pythonExecutionFactory.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -115,15 +115,14 @@ export class PythonExecutionFactory implements IPythonExecutionFactory {
115115
pythonPath: options.interpreter ? options.interpreter.path : undefined,
116116
});
117117
}
118-
// TODO: Try passing actual interpreter when calling this from Pytest
118+
119119
const pythonPath = options.interpreter
120120
? options.interpreter.path
121-
: this.configService.getSettings(options.resource).pythonPath; // : this.configService.getSettings(options.resource).pythonPath; MIGHT BE A PROBLEM.
121+
: this.configService.getSettings(options.resource).pythonPath;
122122
const processService: IProcessService = new ProcessService({ ...envVars });
123123
processService.on('exec', this.logger.logProcess.bind(this.logger));
124124
this.disposables.push(processService);
125125

126-
// TODO: Switch ordering on pixi and conda to see if that helps selecting right conda environment to run. --> Looks like done?
127126
const condaExecutionService = await this.createCondaExecutionService(pythonPath, processService);
128127
if (condaExecutionService) {
129128
return condaExecutionService;

0 commit comments

Comments
 (0)