Skip to content
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 back the PATH known issue #8507

Merged
merged 3 commits into from
Jun 13, 2023
Merged

Conversation

marcpopMSFT
Copy link
Member

@marcpopMSFT marcpopMSFT commented May 31, 2023

We see a couple of customers a month who still hit this. I assume it's because we didn't update 2.1 and that version is still getting installed even by 17.0 and repair could be added the PATH.

https://developercommunity.visualstudio.com/t/The-SDK-MicrosoftNETSdk-specified-co/10372391

We see a couple of customers a month who still hit this. I assume it's because we didn't update 2.1 and that version is still getting installed even by 17.0 and repair could be added the PATH.
Copy link
Member

@leecow leecow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@marcpopMSFT
Copy link
Member Author

The lint error is a false positive as it's flagging a public key token which is not a secret. Not sure what to do with it but we can probably just merge past.

@mairaw
Copy link
Contributor

mairaw commented Jun 8, 2023

The lint error doesn't prevent you from merging. It's just informative.

@marcpopMSFT marcpopMSFT merged commit afcee8f into main Jun 13, 2023
4 of 5 checks passed
@marcpopMSFT marcpopMSFT deleted the marcpopMSFT-addpathknownissue branch June 13, 2023 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants