More .net 7/8 fixes

scottbilas · Nov 22, 2023 · 8d34846 · 8d34846
1 parent 93fe2cd
commit 8d34846
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 8 deletions.
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -21,13 +21,13 @@ jobs:
         with:
           dotnet-version: 7.0.x
 
-      - name: Build for publish
-        run: dotnet publish -c Release
-
       - name: Run tests
         run: |
           dotnet test --nologo -l:"console;verbosity=normal" -c Release
 
+      - name: Pack nuget
+        run: dotnet pack src\Core\Core.csproj -c Release
+
       - name: Publish NuGet package
         run: |
           dotnet nuget push publish\Core\*.nupkg --api-key ${{ secrets.NUGET_API_KEY }} --source https://api.nuget.org/v3/index.json
diff --git a/src/Core/Core-Tests.csproj b/src/Core/Core-Tests.csproj
@@ -1,5 +1,5 @@
 <Project Sdk="Microsoft.NET.Sdk">
-  <PropertyGroup><OkFeatures>T4</OkFeatures></PropertyGroup>
+  <PropertyGroup><OkFeatures>T4;Net78</OkFeatures></PropertyGroup>
   <Import Project="$(OkTargetsRoot)Tests.targets" />
 
   <PropertyGroup>

diff --git a/src/Core/Core.csproj b/src/Core/Core.csproj
@@ -1,5 +1,5 @@
 <Project Sdk="Microsoft.NET.Sdk">
-  <PropertyGroup><OkFeatures>T4</OkFeatures></PropertyGroup>
+  <PropertyGroup><OkFeatures>T4;Net78</OkFeatures></PropertyGroup>
   <Import Project="$(OkTargetsRoot)Library.targets" />
 
   <PropertyGroup>
@@ -8,9 +8,6 @@
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <NoWarn>CS1591</NoWarn>
     <DefineConstants>$(DefineConstants);NICEIO_PUBLIC</DefineConstants>
-
-    <TargetFramework />
-    <TargetFrameworks>net7.0;net8.0</TargetFrameworks>
     <AppendTargetFrameworkToOutputPath>true</AppendTargetFrameworkToOutputPath>
 
     <!-- because it's a package, avoid adding requirements -->

diff --git a/targets/Library.targets b/targets/Library.targets
@@ -58,6 +58,12 @@
     <OkFeatures>;$(OkFeatures);</OkFeatures>
   </PropertyGroup>
 
+  <!--Net78 -->
+  <PropertyGroup Condition=" $(OkFeatures.Contains(';Net78;')) ">
+    <TargetFramework />
+    <TargetFrameworks>net7.0;net8.0</TargetFrameworks>
+  </PropertyGroup>
+
   <!--Core -->
   <ItemGroup Condition=" $(OkFeatures.Contains(';Core;')) or $(OkFeatures.Contains(';Docopt;')) ">
     <ProjectReference Include="$(OkSrcRoot)Core\Core.csproj" />