Skip to content

Commit

Permalink
Fix for issue with environment path and Jedi
Browse files Browse the repository at this point in the history
  • Loading branch information
karthiknadig committed Jan 3, 2024
1 parent 97154eb commit d499a94
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/client/activation/jedi/analysisOptions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ export class JediLanguageServerAnalysisOptions extends LanguageServerAnalysisOpt
},
workspace: {
extraPaths: distinctExtraPaths,
environmentPath: this.interpreter?.envPath,
environmentPath: this.interpreter?.path,
symbols: {
// 0 means remove limit on number of workspace symbols returned
maxSymbols: 0,
Expand Down
2 changes: 1 addition & 1 deletion src/test/activation/jedi/jediAnalysisOptions.unit.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ suite('Jedi LSP - analysis Options', () => {

const result = await analysisOptions.getAnalysisOptions();

expect(result.initializationOptions.workspace.environmentPath).to.deep.equal('.../.venv');
expect(result.initializationOptions.workspace.environmentPath).to.deep.equal('.../.venv/bin/python');
});

test('Without extraPaths provided and no workspace', async () => {
Expand Down

0 comments on commit d499a94

Please sign in to comment.