Skip to content

Set default TFM for Traversal to SDK version #610

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.

Sign up for GitHub

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Set default TFM for Traversal to SDK version #610

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
88783df
Set default TFM for Traversal to SDK version
MattKotsenas Mar 8, 2025
1cc6311
Fix TFM if max version isn't set
MattKotsenas Mar 10, 2025
77ae736
Bump version
MattKotsenas Mar 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion src/Traversal.UnitTests/TraversalTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,8 +204,15 @@ public void PropertiesAreSet(string target)
[InlineData("RestoreProjectStyle", null, "PackageReference")]
[InlineData("StopOnFirstFailure", "false", "false")]
[InlineData("StopOnFirstFailure", null, "true")]
#if NETFRAMEWORK || NET9_0 // These map how MSBuild.ProjectCreation chooses SDK versions given the TFM of the test
[InlineData("TargetFramework", "net6.0", "net6.0")]
[InlineData("TargetFramework", null, "net45")]
[InlineData("TargetFramework", null, "net9.0")]
#elif NET8_0
[InlineData("TargetFramework", "net6.0", "net6.0")]
[InlineData("TargetFramework", null, "net8.0")]
#else
#error "Update test data for the given target framework"
#endif
[InlineData("TraversalProjectNames", "custom.proj", "custom.proj")]
[InlineData("TraversalProjectNames", null, "dirs.proj")]
[InlineData("UsingMicrosoftTraversalSdk", null, "true")]
Expand Down
5 changes: 3 additions & 2 deletions src/Traversal/Sdk/Traversal.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,10 @@
<EnableDefaultItems>false</EnableDefaultItems>

<!--
TargetFramework is required for restore and used to default to .NET Framework v4.5. However, Traversal projects don't specify a version so it needs to be defaulted here.
TargetFramework is required for restore. If the user specifies a version, use that. Otherwise, use the
NETCoreAppMaximumVersion so that the project acts as if it were part of the SDK.
-->
<TargetFramework Condition="'$(TargetFramework)' == ''">net45</TargetFramework>
<TargetFramework Condition=" '$(TargetFramework)' == '' ">net$(NETCoreAppMaximumVersion)</TargetFramework>
</PropertyGroup>

<Import Project="Sdk.targets" Sdk="Microsoft.NET.Sdk" />
Expand Down