Skip to content

Commit 7918328

Browse files
authored
Update telemetry.ts
it looks like in [package.json line 5](https://github.com/microsoft/vscode-cpptools/blob/a62acdb5a52438540c266b409a0ee7f8e64f7a9b/Extension/package.json#L5), we add a suffix, "main", to the version. When we assign customers to target populations for experiments, we were trying to assign customers without any suffix to Public, which resulted in no one being included. Instead, I think we should be checking if the suffix is "main" and assign those customers to Public.
1 parent a62acdb commit 7918328

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Extension/src/telemetry.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ export function activate(): void {
6565
if (packageInfo) {
6666
let targetPopulation: TargetPopulation;
6767
const userVersion: PackageVersion = new PackageVersion(packageInfo.version);
68-
if (userVersion.suffix === "") {
68+
if (userVersion.suffix === "main") {
6969
targetPopulation = TargetPopulation.Public;
7070
} else if (userVersion.suffix === "insiders") {
7171
targetPopulation = TargetPopulation.Insiders;

0 commit comments

Comments
 (0)