Merge pull request #21 from s2quake/ci/improve-action

Improve build and test action

.github/scripts/pack.ps1

@@ -0,0 +1,22 @@
+param (
+    [ValidateScript({ $_ -ge 0 })]
+    [int]$PullRequestNumber = 0,
+    [ValidateScript({ ($_ -eq "") -or (Test-Path $_) })]
+    [string]$KeyPath = ""
+)
+
+$options = @(
+    "-o pack"
+    "-p:FileVersion='$fileVersion'"
+)
+
+if ($PullRequestNumber) {
+    $options += "--version-suffix pr.$PullRequestNumber"
+}
+
+if (Test-Path -Path $KeyPath) {
+    $options += "-p:TreatWarningsAsErrors=true"
+    $options += "-p:AssemblyOriginatorKeyFile='$(Resolve-Path -Path $KeyPath)'"
+}
+Write-Host "dotnet pack $($options -join " ")"
+Invoke-Expression -Command "dotnet pack $($options -join " ")"

.github/workflows/pack.yml

@@ -2,7 +2,7 @@ name: Pack
 
 on:
   pull_request: 
-  
+
 jobs:
   pack:
     runs-on: ubuntu-latest
@@ -12,8 +12,13 @@ jobs:
         with:
           dotnet-version: 8.0.100
       - run: echo "${{ secrets.SNK_FILE }}" | base64 --decode > private.snk
-      - run: |
-          dotnet pack \
-          -o pack \
-          -p:TreatWarningsAsErrors=true \
-          -p:AssemblyOriginatorKeyFile=$(pwd)/private.snk
+      - run: echo "${{ github.event.pull_request.number }}"
+      - run: | 
+          .github/scripts/pack.ps1 `
+          -PullRequestNumber ${{ github.event.pull_request.number }} `
+          -KeyPath "$pwd/private.snk"
+        shell: pwsh
+      - if: env.NUGET_API_KEY != ''
+        run: .github/bin/dist-nuget.sh
+        env:
+          NUGET_API_KEY: ${{ secrets.NUGET_API_KEY }}

Directory.Build.props

@@ -50,7 +50,8 @@ SOFTWARE. -->
     <TargetFrameworks Condition="'$(_IsPacking)'=='true'">net6.0;net7.0;net8.0;netstandard2.1</TargetFrameworks>
     <TargetFramework Condition="'$(_IsPacking)'!='true'">net8.0</TargetFramework>
     <AssemblyVersion>2.0</AssemblyVersion>
-    <FileVersion>2.0.3</FileVersion>
+    <VersionPrefix>2.0.3</VersionPrefix>
+    <FileVersion>$(VersionPrefix)</FileVersion>
 	  <RepositoryType>git</RepositoryType>
 	  <RepositoryUrl>https://github.com/s2quake/communication</RepositoryUrl>
 	  <LicenseUrl>https://github.com/s2quake/communication/blob/main/LICENSE.md</LicenseUrl>

Directory.Build.targets

@@ -29,11 +29,6 @@ SOFTWARE. -->
     <DesiredPublishDirTemp Condition="'$(PublishDir)' != '$(DesiredPublishDir)'">$(PublishDir)</DesiredPublishDirTemp>
     <DesiredPublishDir>$(DesiredPublishDirTemp)</DesiredPublishDir>
   </PropertyGroup>
-  <Target Name="BeforeBuild">
-    <PropertyGroup>
-      <Version>$(FileVersion)-$(TargetFramework)</Version>
-    </PropertyGroup>
-  </Target>
   <Target Name="AfterBuild">
     <PropertyGroup Condition="'$(PublishName)' != ''">
       <PublishDir>$([System.IO.Path]::Combine($(DesiredPublishDir),$(PublishName)))</PublishDir>