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

Fix change done in commit a223146b8c3d5dbbf7bae49149aed762560434c4 #3258

Merged
merged 1 commit into from
Jan 20, 2022

Conversation

Evangelink
Copy link
Member

Description

For some reason, the analyzer was still shown in VS while the condition was broken... The relative path gives us a path that doesn't starts with '/' or '' so the condition looking for '/src' or '\src' was always false. I removed the first '/' or '' and updated the condition to be starts with instead of contains as we are now sure to always look from the root folder of the repo.

@nohwnd nohwnd enabled auto-merge (squash) January 17, 2022 15:35
@nohwnd nohwnd merged commit 82235d6 into microsoft:main Jan 20, 2022
@Evangelink Evangelink deleted the fix-publicapi-condition branch January 20, 2022 09:59
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.

2 participants