Current we use boolean values to determine whether os is one of Mac, Windows or Linux.
Should consider using an Enum.
This could result in a number of changes, however I believe it is worth, and needs to be done.
Using 3 booleans doesn't work well, when all of them are mutually exclusive.
Suggestion:
export enum OS {
Linux = 'Linux',
Mac = 'Mac',
Windows = 'Windows'
}
@Microsoft/pvsc-team
Current we use boolean values to determine whether os is one of Mac, Windows or Linux.
Should consider using an Enum.
This could result in a number of changes, however I believe it is worth, and needs to be done.
Using 3 booleans doesn't work well, when all of them are mutually exclusive.
Suggestion:
@Microsoft/pvsc-team