From bcec7bae0768cf9b3c9715fce3f908437752524b Mon Sep 17 00:00:00 2001
From: Matthew de Detrich <matthew.dedetrich@aiven.io>
Date: Tue, 21 Feb 2023 15:42:16 +0100
Subject: [PATCH] Move header check into its own github action

---
 .github/workflows/build-test-prValidation.yml | 29 ------------------
 .github/workflows/headers.yml                 | 30 +++++++++++++++++++
 project/CopyrightHeader.scala                 |  9 ++----
 3 files changed, 32 insertions(+), 36 deletions(-)
 create mode 100644 .github/workflows/headers.yml

diff --git a/.github/workflows/build-test-prValidation.yml b/.github/workflows/build-test-prValidation.yml
index d5b9c0ae07..d693b7d11a 100644
--- a/.github/workflows/build-test-prValidation.yml
+++ b/.github/workflows/build-test-prValidation.yml
@@ -11,35 +11,6 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  check-headers:
-    name: Check / Headers
-    runs-on: ubuntu-20.04
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v3
-        with:
-          # we don't know what commit the last tag was it's safer to get entire repo so previousStableVersion resolves
-          fetch-depth: 0
-
-      - name: Setup Java 8
-        uses: actions/setup-java@v3
-        with:
-          distribution: temurin
-          java-version: 8
-
-      - name: Cache Coursier cache
-        uses: coursier/cache-action@v6.4.0
-
-      - name: Enable jvm-opts
-        run: cp .jvmopts-ci .jvmopts
-
-      - name: Check headers
-        run: |-
-          sbt \
-          -Dsbt.override.build.repos=false \
-          -Dsbt.log.noformat=false \
-          +headerCheckAll
-
   pull-request-validation:
     name: Check / Tests
     runs-on: ubuntu-20.04
diff --git a/.github/workflows/headers.yml b/.github/workflows/headers.yml
new file mode 100644
index 0000000000..6eda4f3a31
--- /dev/null
+++ b/.github/workflows/headers.yml
@@ -0,0 +1,30 @@
+name: Headers
+
+on:
+  pull_request:
+
+permissions: {}
+
+jobs:
+  check-headers:
+    name: Check headers
+    runs-on: ubuntu-20.04
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Setup Java 8
+        uses: actions/setup-java@v3
+        with:
+          distribution: temurin
+          java-version: 8
+
+      - name: Cache Coursier cache
+        uses: coursier/cache-action@v6.4.0
+
+      - name: Check headers
+        run: |-
+          sbt \
+          -Dsbt.override.build.repos=false \
+          -Dsbt.log.noformat=false \
+          +headerCheckAll
diff --git a/project/CopyrightHeader.scala b/project/CopyrightHeader.scala
index 274e6fb2e8..4e01c4f6f2 100644
--- a/project/CopyrightHeader.scala
+++ b/project/CopyrightHeader.scala
@@ -13,7 +13,6 @@
 
 package org.apache.pekko
 
-import org.apache.pekko.PekkoValidatePullRequest.additionalTasks
 import de.heikoseeberger.sbtheader.HeaderPlugin.autoImport._
 import de.heikoseeberger.sbtheader.{ CommentCreator, HeaderPlugin, NewLine }
 import com.typesafe.sbt.MultiJvmPlugin.MultiJvmKeys._
@@ -38,7 +37,7 @@ trait CopyrightHeader extends AutoPlugin {
             HeaderFileType("template") -> cStyleComment)))
     }
 
-  override def projectSettings: Seq[Def.Setting[_]] = Def.settings(headerMappingSettings, additional)
+  override def projectSettings: Seq[Def.Setting[_]] = headerMappingSettings
 
   def additional: Seq[Def.Setting[_]] =
     Def.settings(Compile / compile := {
@@ -105,8 +104,4 @@ trait CopyrightHeader extends AutoPlugin {
 
 object CopyrightHeader extends CopyrightHeader
 
-object CopyrightHeaderInPr extends CopyrightHeader {
-
-  override val additional =
-    Def.settings(additionalTasks += Compile / headerCheck, additionalTasks += Test / headerCheck)
-}
+object CopyrightHeaderInPr extends CopyrightHeader