Skip to content

Conversation

@Faizan2304
Copy link
Contributor

Issue: #608
Fix: Inject entry point only for project targeting netcoreapp

Fix: Inject entry point only for project targeting netcoreapp1.0
@msftclas
Copy link

@Faizan2304,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Copy link
Contributor

@harshjain2 harshjain2 left a comment

Choose a reason for hiding this comment

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

Suggestion : Maintain two separate copies for Microsoft.NET.Test.Sdk.targets for dotnet core and full clr Target Frameworks, This will avoid issues like #608 in future.

============================================================
-->
<PropertyGroup>
<PropertyGroup Condition="'$(TargetFrameworkIdentifier)' == '.NETCoreApp'">
Copy link
Contributor

Choose a reason for hiding this comment

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

A general thought (not specific to this PR):
For every build of a netcore test project, we will create this file. Does this imply that the test project will be compiled again always? (even if no actual code has changed)

If this is true, can it be avoided?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, It will create only first time. Subsequent build and rebuild will use the same

@Faizan2304 Faizan2304 merged commit 7219981 into microsoft:master Mar 28, 2017
@Faizan2304 Faizan2304 deleted the EntryPoint branch March 28, 2017 06:09
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.

4 participants