Combine and separate workflows

- Separate between PR and Push execution (with and without secrets)
- Turn tests into re-usable workflows and call them from the main ones

Author: softworkz

.github/workflows/ci.yml .github/workflows/Build and Publish.yml.github/workflows/ci.yml renamed to .github/workflows/Build and Publish.yml

@@ -1,31 +1,25 @@
-name: CI
+name: Build and Publish
 
-on: [push, pull_request]
+on: [push]
 
 env:
   GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
   NUGET_API_KEY: ${{ secrets.NUGET_API_KEY }}
 
-jobs:
-  # linux:
-  #   runs-on: ubuntu-latest
-  #   timeout-minutes: 10
-
-  #   steps:
-  #   - uses: actions/checkout@v4
-
-  #   - name: Setup dotnet
-  #     uses: actions/setup-dotnet@v4
-  #     with:
-  #       dotnet-version: |
-  #         6.0.x
-  #         8.0.x
-  #         10.0.x
+concurrency:
+  group: build-publish-${{ github.ref }}
+  cancel-in-progress: true
 
-  #   - name: Build
-  #     run: ./build.sh
-
-  windows:
+jobs:
+  whitespace-check:
+    uses: ./.github/workflows/trailing-whitespace-check.yml
+    
+  integration-tests:
+    needs: whitespace-check
+    uses: ./.github/workflows/integration-tests.yml
+
+  publish:
+    needs: [whitespace-check, integration-tests]
     runs-on: windows-latest
     timeout-minutes: 10
 

.github/workflows/PR Validation.yml

@@ -0,0 +1,36 @@
+name: PR Validation
+
+on: [pull_request]
+
+concurrency:
+  group: pr-validation-${{ github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  whitespace-check:
+    uses: ./.github/workflows/trailing-whitespace-check.yml
+    secrets: inherit
+    
+  integration-tests:
+    needs: whitespace-check
+    uses: ./.github/workflows/integration-tests.yml
+    secrets: inherit
+
+  build:
+    needs: [whitespace-check, integration-tests]
+    runs-on: windows-latest
+    timeout-minutes: 10
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Setup dotnet
+      uses: actions/setup-dotnet@v4
+      with:
+        dotnet-version: |
+          6.0.x
+          8.0.x
+          10.0.x
+
+    - name: Build
+      run: .\build.ps1

.github/workflows/integration-tests.yml

@@ -1,14 +1,12 @@
 name: Tests
 
 on:
-  push:
-    branches: [ develop, main ]
-  pull_request:
-    branches: [ develop, main ]
-
-concurrency:
-  group: integration-tests-${{ github.ref }}
-  cancel-in-progress: true
+  workflow_call:
+    permissions:
+      actions: read
+      contents: read
+      checks: write
+      pull-requests: write
 
 jobs:
   tests:

.github/workflows/trailing-whitespace-check.yml

@@ -1,8 +1,12 @@
 name: Trailing Whitespace Check
 
 on:
-  pull_request:
-    types: [opened, synchronize, reopened]
+  workflow_call:
+    permissions:
+      actions: read
+      contents: read
+      checks: write
+      pull-requests: write
 
 jobs:
   check-trailing-whitespace:

src/ElectronNET.sln

@@ -41,10 +41,12 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Test Apps", "Test Apps", "{
 EndProject
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Build", "Build", "{985D39A7-5216-4945-8167-2FD0CB387BD8}"
 	ProjectSection(SolutionItems) = preProject
-		..\.github\workflows\ci.yml = ..\.github\workflows\ci.yml
+		..\.github\workflows\Build and Publish.yml = ..\.github\workflows\Build and Publish.yml
 		..\.github\workflows\integration-tests.yml = ..\.github\workflows\integration-tests.yml
+		..\.github\workflows\PR Validation.yml = ..\.github\workflows\PR Validation.yml
 		..\.github\workflows\pr-comment.yml = ..\.github\workflows\pr-comment.yml
 		..\.github\workflows\publish-wiki.yml = ..\.github\workflows\publish-wiki.yml
+		..\.github\workflows\retry-test-jobs.yml.yml = ..\.github\workflows\retry-test-jobs.yml.yml
 		..\.github\workflows\trailing-whitespace-check.yml = ..\.github\workflows\trailing-whitespace-check.yml
 	EndProjectSection
 EndProject