-
Notifications
You must be signed in to change notification settings - Fork 1.2k
removed experiment code and extra setting #13900
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
Conversation
Did we decide this experiment was a success? And nobody ever turned the setting on? |
How else is somebody going to change a kernel on a remote machine? Or switch to say .NET or Julia kernel? |
Right, it was only supposed to disappear on local machines |
I'll check telemetry for the setting on Monday, but I'm sure there were no issues about this. |
is not local
Codecov Report
@@ Coverage Diff @@
## main #13900 +/- ##
==========================================
- Coverage 59.87% 59.84% -0.03%
==========================================
Files 693 693
Lines 38348 38345 -3
Branches 5515 5514 -1
==========================================
- Hits 22961 22949 -12
- Misses 14207 14213 +6
- Partials 1180 1183 +3
Continue to review full report at Codecov.
|
Kudos, SonarCloud Quality Gate passed!
|
@rchiodo can confirm, no one ever turned the setting on. |
package-lock.json
has been regenerated by runningnpm install
(if dependencies have changed).