Build and release action improvements

* Let Build.ps1 and release.yml run on an ubuntu agent
* Full automatic release: use last commit message (usually merge commit from PR) as release description

Author: ckadluba

.github/workflows/release.yml

@@ -2,40 +2,56 @@ name: Release
 
 on:
   push:
-    branches: [ dev, main ]
+    branches:
+      - main
+      - dev
 
   # Allows you to run this workflow manually from the Actions tab
   workflow_dispatch:
 
 jobs:
   build-and-release:
-    runs-on: windows-latest
+    runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Read version from csproj
+        id: extract_version
+        if: github.ref == 'refs/heads/main'
+        run: |
+          VERSION=$(grep '<VersionPrefix>' src/Serilog.Sinks.MSSqlServer/Serilog.Sinks.MSSqlServer.csproj | sed 's/.*<VersionPrefix>\(.*\)<\/VersionPrefix>.*/\1/')
+          echo "VERSION=$VERSION" >> $GITHUB_ENV
+          if git rev-parse "v$VERSION" >/dev/null 2>&1; then
+            echo "Tag v$VERSION already exists"
+            exit 1
+          fi
 
       - name: Run build
         run: ./Build.ps1 -SkipTests
+        shell: pwsh
 
-      - name: Read version from csproj
+      - name: Get last commit message
+        id: last_commit
+        if: success() && github.ref == 'refs/heads/main'
         run: |
-          $selectResult = Select-String -Path ".\src\Serilog.Sinks.MSSqlServer\Serilog.Sinks.MSSqlServer.csproj" -Pattern '<VersionPrefix>(.+)</VersionPrefix>'
-          $versionMatch = $selectResult.Matches[0].Groups[1].Value
-          echo ("VERSION=" + $versionMatch) >> $env:GITHUB_ENV
-          echo "Found version $versionMatch"
-
-      - name: Create GitHub release
-        if: ${{ github.ref_name == 'main' }}
-        uses: "marvinpinto/action-automatic-releases@latest"
-        with:
-          repo_token: "${{ secrets.GITHUB_TOKEN }}"
-          prerelease: false
-          automatic_release_tag: "v${{ env.VERSION }}"
-          title: "v${{ env.VERSION }}"
-          files: |
-            artifacts/*.nupkg
-            artifacts/*.snupkg
+          git log -1 --pretty=%B > last_commit_message.txt
+
+      # Create GitHub release only on main branch (latest release)
+      - name: Create Release
+        if: github.ref == 'refs/heads/main' && success()
+        env:
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        run: |
+          gh release create v${{ env.VERSION }} \
+            --title "v${{ env.VERSION }}" \
+            --notes "$(cat last_commit_message.txt)" \
+            artifacts/*.nupkg artifacts/*.snupkg
 
       - name: Publish to nuget.org
         env:
           NUGET_API_KEY: ${{ secrets.NUGET_API_KEY }}
-        run: nuget push artifacts\*.nupkg -Source 'https://api.nuget.org/v3/index.json' -ApiKey ${{ env.NUGET_API_KEY }}
+        run: |
+          nuget push artifacts/*.nupkg -Source https://api.nuget.org/v3/index.json -ApiKey ${{ secrets.NUGET_API_KEY }}
+          nuget push artifacts/*.snupkg -Source https://api.nuget.org/v3/index.json -ApiKey ${{ secrets.NUGET_API_KEY }}

Build.ps1

@@ -7,55 +7,55 @@ param (
 
 echo "build: Build started"
 
-Push-Location $PSScriptRoot
+Push-Location "$PSScriptRoot"
 
-if(Test-Path .\artifacts) {
-	echo "build: Cleaning .\artifacts"
-	Remove-Item .\artifacts -Force -Recurse
+if (Test-Path .\artifacts) {
+    echo "build: Cleaning .\artifacts"
+    Remove-Item .\artifacts -Force -Recurse
 }
 
 & dotnet restore --no-cache
 
-$branch = @{ $true = $env:GITHUB_REF_NAME; $false = $(git symbolic-ref --short -q HEAD) }[$env:GITHUB_REF_NAME -ne $NULL];
-$revision = @{ $true = "{0:00000}" -f [convert]::ToInt32("0" + $env:GITHUB_RUN_NUMBER, 10); $false = "local" }[$env:GITHUB_RUN_NUMBER -ne $NULL];
-$suffix = @{ $true = ""; $false = "$($branch.Substring(0, [math]::Min(10,$branch.Length)))-$revision"}[$branch -ne "dev" -and $revision -ne "local"]
+$branch = @{ $true = $env:GITHUB_REF_NAME; $false = $(git symbolic-ref --short -q HEAD) }[$env:GITHUB_REF_NAME -ne $NULL]
+$revision = @{ $true = "{0:00000}" -f [convert]::ToInt32("0" + $env:GITHUB_RUN_NUMBER, 10); $false = "local" }[$env:GITHUB_RUN_NUMBER -ne $NULL]
+$suffix = @{ $true = ""; $false = "$($branch.Substring(0, [math]::Min(10, $branch.Length)))-$revision" }[$branch -ne "dev" -and $revision -ne "local"]
 
 echo "build: Version suffix is $suffix"
 
-foreach ($src in ls src/*) {
-    Push-Location $src
+foreach ($src in Get-ChildItem "$PSScriptRoot/src" -Directory) {
+    Push-Location $src.FullName
 
-    echo "build: Packaging project in $src"
+    echo "build: Packaging project in $($src.FullName)"
 
     if ($suffix) {
         & dotnet pack -c Release -o ..\..\artifacts --version-suffix=$suffix
     } else {
         & dotnet pack -c Release -o ..\..\artifacts
     }
-    if($LASTEXITCODE -ne 0) { exit 1 }
+    if ($LASTEXITCODE -ne 0) { exit 1 }
 
     Pop-Location
 }
 
 if ($SkipTests -eq $false) {
-    foreach ($test in ls test/*.PerformanceTests) {
-        Push-Location $test
+    foreach ($test in Get-ChildItem "$PSScriptRoot/test" -Filter "*.PerformanceTests" -Directory) {
+        Push-Location $test.FullName
 
-        echo "build: Building performance test project in $test"
+        echo "build: Building performance test project in $($test.FullName)"
 
         & dotnet build -c Release
-        if($LASTEXITCODE -ne 0) { exit 2 }
+        if ($LASTEXITCODE -ne 0) { exit 2 }
 
         Pop-Location
     }
 
-    foreach ($test in ls test/*.Tests) {
-        Push-Location $test
+    foreach ($test in Get-ChildItem "$PSScriptRoot/test" -Filter "*.Tests" -Directory) {
+        Push-Location $test.FullName
 
-        echo "build: Testing project in $test"
+        echo "build: Testing project in $($test.FullName)"
 
         & dotnet test -c Release --collect "XPlat Code Coverage"
-        if($LASTEXITCODE -ne 0) { exit 3 }
+        if ($LASTEXITCODE -ne 0) { exit 3 }
 
         Pop-Location
     }