You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pekko.apache.org by md...@apache.org on 2023/02/21 17:21:37 UTC
[incubator-pekko] branch main updated: Move header check into its own github action
This is an automated email from the ASF dual-hosted git repository.
mdedetrich pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-pekko.git
The following commit(s) were added to refs/heads/main by this push:
new bcec7bae07 Move header check into its own github action
bcec7bae07 is described below
commit bcec7bae0768cf9b3c9715fce3f908437752524b
Author: Matthew de Detrich <ma...@aiven.io>
AuthorDate: Tue Feb 21 15:42:16 2023 +0100
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(-)
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
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pekko.apache.org
For additional commands, e-mail: commits-help@pekko.apache.org