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

Improve build and test action

.github/scripts/pack.ps1

@@ -0,0 +1,35 @@
+#!/usr/local/bin/pwsh
+param (
+    [ValidateScript({ ($_ -eq "") -or !(Test-Path $_ -PathType Leaf) })]
+    [string]$OutputPath = "",
+    [ValidateScript({ $_ -ge 0 })]
+    [int]$PullRequestNumber = 0,
+    [ValidateScript({ ($_ -eq "") -or (Test-Path $_) })]
+    [string]$KeyPath = ""
+)
+
+$namespaces = @{
+    ns = "http://schemas.microsoft.com/developer/msbuild/2003"
+}
+$propsPath = "Directory.Build.props"
+$fileVersionPath = "/ns:Project/ns:PropertyGroup/ns:FileVersion"
+$result = Select-Xml -Path $propsPath -Namespace $namespaces -XPath $fileVersionPath
+if ($null -eq $result) {
+    Write-Host "File version not found"
+    exit 1
+}
+
+$fileVersion = $result.Node.InnerText
+$KeyPath = $KeyPath ? $(Resolve-Path -Path $KeyPath) : ""
+$OutputPath = $OutputPath ? [System.IO.Path]::GetFullPath($OutputPath) : ""
+$keyPathExits = Test-Path -Path $KeyPath
+
+$options = @(
+    $OutputPath ? "-o '$OutputPath'" : ""
+    "-p:FileVersion='$fileVersion'"
+    $PullRequestNumber ? "--version-suffix pr.$PullRequestNumber" : ""
+    $keyPathExits ? "-p:TreatWarningsAsErrors=true" : ""
+    $keyPathExits ? "-p:AssemblyOriginatorKeyFile='$KeyPath'" : ""
+) | Where-Object { $_ }
+
+Invoke-Expression -Command "dotnet pack $($options -join " ")"

.github/workflows/pack.yml

@@ -2,7 +2,12 @@ name: Pack
 
 on:
   pull_request: 
-
+    push:
+    branches:
+      - main
+      tags:
+        - "*"
+
 jobs:
   pack:
     runs-on: ubuntu-latest
@@ -12,8 +17,9 @@ 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
+      - if: ${{ github.event_name == 'pull_request' }}
+        run: | 
+          .github/scripts/pack.ps1 `
+          -PullRequestNumber ${{ github.event.pull_request.number }} `
+          -KeyPath "$pwd/private.snk"
+        shell: pwsh

Directory.Build.props

@@ -57,7 +57,8 @@ SOFTWARE. -->
 	  <Copyright>Copyright (c) 2024 Jeesu Choi</Copyright>
 	  <Description>grpc-based communication library for crema project</Description>
 	  <Authors>s2quake</Authors>
-	  <PackageVersion>$(FileVersion)</PackageVersion>
+	  <PackageVersion Condition="'$(VersionSuffix)'==''">$(FileVersion)</PackageVersion>
+	  <PackageVersion Condition="'$(VersionSuffix)'!=''">$(FileVersion)-$(VersionSuffix)</PackageVersion>
     <PackageProjectUrl>https://github.com/s2quake/communication</PackageProjectUrl>
     <PackageLicenseFile>LICENSE.md</PackageLicenseFile>
     <PackageReadmeFile>README.md</PackageReadmeFile>

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>