3 ms·
If the message is to be believed (and the source code does agree: https://github.com/microsoft/vscode/blob/d32b92bd7a49ce8667b954d86320cc29545fc505/src/vs/platf
by rcxdude 4y ago
If the message is to be believed (and the source code does agree: https://github.com/microsoft/vscode/blob/d32b92bd7a49ce8667b954d86320cc29545fc505/src/vs/platform/telemetry/common/telemetryUtils.ts#L125 https://github.com/microsoft/vscode/blob/d32b92bd7a49ce8667b...), the old settings still apply, the new one may default to 'all' but the old settings will override it, so they've avoided turning telemetry back on for those who had disabled it (in fact if you didn't update the settings and for some reason had only turned off one of the telemetry options, they have in fact reduced the amount of telemetry they are getting from you).
- alberth 4y ago> @returns OFF, ERROR, ON > ... > return TelemetryLevel.NONE; You gotta love how the documented function return value (TelemetryConfiguration.OFF) doesn't match the actual return value (TelemetryConfiguration.NONE).
- solardev 4y agoIf only there was some typing system that could help with this.