- Notifications
You must be signed in to change notification settings - Fork1.2k
More specific llm tools#25072
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
More specific llm tools#25072
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| }, | ||
| "when": "!pythonEnvExtensionInstalled" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Will disable listPythonPackages for now
| @@ -17,6 +17,7 @@ suite('Python Path Settings Updater', () => { | |||
| serviceContainer = TypeMoq.Mock.ofType<IServiceContainer>(); | |||
| workspaceService = TypeMoq.Mock.ofType<IWorkspaceService>(); | |||
| interpreterPathService = TypeMoq.Mock.ofType<IInterpreterPathService>(); | |||
| experimentsManager = TypeMoq.Mock.ofType<IExperimentService>(); | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Fix some errors in other parts of the code that ws causing issues
| @@ -36,6 +37,11 @@ suite('Execution Flow Run Adapters', () => { | |||
| let useEnvExtensionStub: sinon.SinonStub; | |||
| setup(() => { | |||
| const proc = typeMoq.Mock.ofType<MockChildProcess>(); | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Fix some errors in other parts of the code that ws causing issues
| @@ -62,7 +62,7 @@ export async function updateSetting( | |||
| configTarget: ConfigurationTarget, | |||
| ) { | |||
| const vscode = require('vscode') as typeof import('vscode'); | |||
| const settings = vscode.workspace.getConfiguration('python', { uri: resource, languageId: 'python' } || null); | |||
| const settings = vscode.workspace.getConfiguration('python', { uri: resource, languageId: 'python' }); | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
|| null was not required, as its impossible for object to be empty.
was getting these errors locally.
5045cdd intomainUh oh!
There was an error while loading.Please reload this page.
No description provided.