Skip to content

Detect whether a project file is a dynamic library #8099

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

Merged
merged 4 commits into from
Jun 26, 2021

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Jun 24, 2021

Resolves #8072

Microsoft Reviewers: Open in CodeFlow

@tido64 tido64 requested review from NickGerleman and asklar June 24, 2021 06:30
@tido64 tido64 requested a review from a team as a code owner June 24, 2021 06:30
@tido64 tido64 force-pushed the tido/exclude-application-projects branch from c1f4bdc to 4de051f Compare June 24, 2021 06:54
@tido64 tido64 force-pushed the tido/exclude-application-projects branch from 4de051f to b93a2fd Compare June 24, 2021 10:32
@NickGerleman
Copy link
Contributor

@tido64 FYI looks like this is causing failures in auto-linking UTs.

@tido64
Copy link
Member Author

tido64 commented Jun 24, 2021

@tido64 FYI looks like this is causing failures in auto-linking UTs.

Thanks, I will take a look later. I got pulled into another thing 😄

@asklar asklar requested a review from jonthysell June 25, 2021 07:25
@asklar
Copy link
Member

asklar commented Jun 25, 2021

@tido64 consider adding a unit test to make sure we don't regress this again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Autolinking is picking up project files not meant to be autolinked
3 participants