-
Notifications
You must be signed in to change notification settings - Fork 302
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
Add run notebook in dedicated extension host command #9492
Conversation
src/client/common/serviceRegistry.ts
Outdated
@@ -88,6 +89,10 @@ export function registerTypes(serviceManager: IServiceManager) { | |||
IExtensionSingleActivationService, | |||
ReloadVSCodeCommandHandler | |||
); | |||
serviceManager.addSingleton<IExtensionSingleActivationService>( |
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.
You must have older bits. This folder should exist anymore.
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.
oh I see, let me rebase
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.
Package-lock.json shouldn't be updated. Ian's going to do that after the release, but it looks like your running of npm install updated it.
My update from main is in now. I think that at this point we want to go ahead and update package-lock.json. Now that release is branched off it's safe to do so. It would just need to merge due to my package-lock changes. |
Codecov Report
@@ Coverage Diff @@
## main #9492 +/- ##
======================================
Coverage 73% 73%
======================================
Files 191 229 +38
Lines 8854 10524 +1670
Branches 1303 1541 +238
======================================
+ Hits 6482 7746 +1264
- Misses 1857 2149 +292
- Partials 515 629 +114
|
import { ICommandManager, IWorkspaceService } from '../types'; | ||
|
||
/** | ||
* Prompts user to reload VS Code with a custom message, and reloads if necessary. |
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.
Does it actually prompt? I don't see that happening in the code
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.
not sure why the package-lock is changing.
we have a debt item to address this (@rchiodo @IanMatthewHuff Ajay aware of this issue) please could you revert the changes to the package lock file
Updated the package lock. Something is off with linting still. Looking into it quick. |
Small tslint fix on the affinity max check. |
Fixes microsoft/vscode#140374
package-lock.json
has been regenerated by runningnpm install
(if dependencies have changed).