-
Notifications
You must be signed in to change notification settings - Fork 42
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
Warnings on <Sdk /> style import #97
Comments
Hi, The extras should be in place of the Microsoft.NET.Sdk in the Project element. It will include that in the correct locations. |
@onovotny of course I know that. The issue is about support of almost identical method: |
That cannot work though due to ordering requirements. The extras needs things both before and after the main SDK targets to work. Thats why it does the import. |
I see. Makes sense. Still, that's unfortunate limitation. Linking msbuild issue for reference. Feel free to close this if you believe this is not an issue then. |
MSBuild warnings:
Sample project:
I believe some checks are needed in Sdk\Sdk.props and Sdk\Sdk.targets not to import Microsoft.NET.Sdk if already imported, since
<Sdk />
imports are valid scenario too (I believe they are much more maintainable than<Project Sdk="" />
).The text was updated successfully, but these errors were encountered: