-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix raw kernel running other languages #12111
Conversation
Fix connection file clashing
Duh I forgot to mention, I also made it possible to switch kernels without having a notebook open. This allows the user to actually switch to a kernel that works in case the one in the notebook won't start. |
@@ -46,11 +47,19 @@ export class NotebookProvider implements INotebookProvider { | |||
disposables.push( | |||
interactiveWindowProvider.onDidChangeActiveInteractiveWindow(this.checkAndDisposeNotebook, this) | |||
); | |||
// this.rawNotebookProvider |
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.
Doh. This should be fixed.
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.
This is used if the initial kernel doesn't run. After I fixed the kernel running I didn't hit this again.
In reply to: 434219118 [](ancestors = 434219118)
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.
Maybe I should write a test that mimics a failure in the kernel on startup.
In reply to: 434219880 [](ancestors = 434219880,434219118)
src/test/datascience/jupyter/kernels/kernelSwitcher.unit.test.ts
Outdated
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
For #11919
This finishes the work to support other languages. Note, there's no tests for it. (other than some basic unit tests for kernel picking). We don't install dotnet on the azure machine nor do we test the language tokenization.
I feel like this is okay at this time as we're not officially supporting any other languages yet.
I'm going to add another issue for 'testing' other languages.
https://github.com/microsoft/vscode-python/issues/12112