Update workflow skippers, nd add style check workflow (#49)

[skip version] [skip validation] [skip style] [skip spelling]
insightsengineering · Jan 29, 2022 · ca64463 · ca64463
1 parent 6a3f558
commit ca64463
Show file tree

Hide file tree

Showing 17 changed files with 54 additions and 16 deletions.
diff --git a/.github/workflows/audit.yaml b/.github/workflows/audit.yaml
@@ -17,7 +17,7 @@ jobs:
     container:
       image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
     name: oysteR scan 🦪
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip audit]')"
+    if: "! contains(github.event.commits[0].message, '[skip audit]')"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/.github/workflows/bioccheck.yaml b/.github/workflows/bioccheck.yaml
@@ -50,7 +50,7 @@ jobs:
             tag: latest
     name: ${{ matrix.config.image }}, version ${{ matrix.config.tag }}
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip bioccheck]') && contains(inputs.enable-bioccheck, 'true')"
+    if: "! contains(github.event.commits[0].message, '[skip bioccheck]') && contains(inputs.enable-bioccheck, 'true')"
     container:
       image: ${{ matrix.config.image }}:${{ matrix.config.tag }}
 

diff --git a/.github/workflows/build-check-install.yaml b/.github/workflows/build-check-install.yaml
@@ -40,7 +40,7 @@ jobs:
             tag: latest
     name: ${{ matrix.config.image }}, version ${{ matrix.config.tag }}
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip r-cmd]')"
+    if: "! contains(github.event.commits[0].message, '[skip r-cmd]')"
     container:
       image: ${{ matrix.config.image }}:${{ matrix.config.tag }}
 
@@ -155,7 +155,7 @@ jobs:
     name: Upload build tar.gz
     needs: build-install-check
     runs-on: ubuntu-latest
-    if: "startsWith(github.ref, 'refs/tags/v') && (! contains(toJSON(github.event.commits.*.message), '[skip r-cmd]'))"
+    if: "startsWith(github.ref, 'refs/tags/v') && (! contains(github.event.commits[0].message, '[skip r-cmd]'))"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/.github/workflows/gitleaks.yaml b/.github/workflows/gitleaks.yaml
@@ -22,7 +22,7 @@ jobs:
   gitleaks:
     name: gitleaks 💧
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip gitleaks]')"
+    if: "! contains(github.event.commits[0].message, '[skip gitleaks]')"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/.github/workflows/licenses.yaml b/.github/workflows/licenses.yaml
@@ -15,7 +15,7 @@ jobs:
   license:
     runs-on: ubuntu-latest
     name: Check 🚩
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip licenses]')"
+    if: "! contains(github.event.commits[0].message, '[skip licenses]')"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/.github/workflows/links.yaml b/.github/workflows/links.yaml
@@ -22,7 +22,7 @@ jobs:
   linkChecker:
     name: Validate Links 🔎
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip links]')"
+    if: "! contains(github.event.commits[0].message, '[skip links]')"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/.github/workflows/linter.yaml b/.github/workflows/linter.yaml
@@ -16,7 +16,7 @@ jobs:
   lint:
     name: SuperLinter 🦸‍♂️
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip linter]')"
+    if: "! contains(github.event.commits[0].message, '[skip linter]')"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/.github/workflows/pkgdown.yaml b/.github/workflows/pkgdown.yaml
@@ -46,7 +46,7 @@ jobs:
   pkgdown:
     name: Generate 🐣
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip docs]')"
+    if: "! contains(github.event.commits[0].message, '[skip docs]')"
     container:
       image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
     steps:

diff --git a/.github/workflows/release.yaml b/.github/workflows/release.yaml
@@ -11,7 +11,7 @@ jobs:
   release:
     name: Release 🚀
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip release]')"
+    if: "! contains(github.event.commits[0].message, '[skip release]')"
     permissions:
       contents: write
     steps:

diff --git a/.github/workflows/roxygen.yaml b/.github/workflows/roxygen.yaml
@@ -32,7 +32,7 @@ jobs:
   man-pages:
     name: Manual pages check 🏁
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip roxygen]')"
+    if: "! contains(github.event.commits[0].message, '[skip roxygen]')"
     container:
       image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
 

diff --git a/.github/workflows/spelling.yaml b/.github/workflows/spelling.yaml
@@ -16,7 +16,7 @@ jobs:
   spelling:
     name: Check spelling 🔠
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip spelling]')"
+    if: "! contains(github.event.commits[0].message, '[skip spelling]')"
     container:
       image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
 

diff --git a/.github/workflows/style.yaml b/.github/workflows/style.yaml
@@ -0,0 +1,36 @@
+---
+name: Style 🎽
+
+on:
+  push:
+    branches:
+      - main
+      - pre-release
+  pull_request:
+    branches:
+      - main
+      - pre-release
+  workflow_call:
+
+jobs:
+  style:
+    name: Check code style 🔠
+    runs-on: ubuntu-latest
+    if: "! contains(github.event.commits[0].message, '[skip style]')"
+    container:
+      image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
+
+    steps:
+      - name: Checkout Code 🛎
+        uses: actions/checkout@v2
+        with:
+          path: ${{ github.event.repository.name }}
+
+      - name: Install styler 👚
+        run: |
+          if (!require("styler")) install.packages("styler", repos="http://cran.rstudio.com/")
+        shell: Rscript {0}
+
+      - name: Run styler 👟
+        run: styler::style_pkg("${{ github.event.repository.name }}", dry = "fail")
+        shell: Rscript {0}
diff --git a/.github/workflows/test-coverage.yaml b/.github/workflows/test-coverage.yaml
@@ -32,7 +32,7 @@ jobs:
   test-coverage:
     name: Coverage 📔
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip coverage]')"
+    if: "! contains(github.event.commits[0].message, '[skip coverage]')"
     container:
       image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
     steps:

diff --git a/.github/workflows/validation.yaml b/.github/workflows/validation.yaml
@@ -33,7 +33,7 @@ jobs:
   r-pkg-validation:
     name: Create report 📃
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip validation]')"
+    if: "! contains(github.event.commits[0].message, '[skip validation]')"
     container:
       image: ghcr.io/insightsengineering/rstudio_4.1.2_bioc_3.14:latest
     env:

diff --git a/.github/workflows/version-bump.yaml b/.github/workflows/version-bump.yaml
@@ -16,7 +16,7 @@ jobs:
   version-bump:
     name: Bump version ⤴
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip vbump]')"
+    if: "! contains(github.event.commits[0].message, '[skip vbump]')"
     container:
       image: docker.io/rocker/tidyverse:4.1.2
 

diff --git a/.github/workflows/version.yaml b/.github/workflows/version.yaml
@@ -16,7 +16,7 @@ jobs:
   version-check:
     name: Version check 🏁
     runs-on: ubuntu-latest
-    if: "! contains(toJSON(github.event.commits.*.message), '[skip version]')"
+    if: "! contains(github.event.commits[0].message, '[skip version]')"
     steps:
       - name: Checkout repo 🛎
         uses: actions/checkout@v2

diff --git a/README.md b/README.md
@@ -11,6 +11,7 @@
 [![Release](https://github.com/insightsengineering/r.pkg.template/actions/workflows/release.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/release.yaml)
 [![Roxygen](https://github.com/insightsengineering/r.pkg.template/actions/workflows/roxygen.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/roxygen.yaml)
 [![Spelling](https://github.com/insightsengineering/r.pkg.template/actions/workflows/spelling.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/spelling.yaml)
+[![Style](https://github.com/insightsengineering/r.pkg.template/actions/workflows/style.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/style.yaml)
 [![SuperLinter](https://github.com/insightsengineering/r.pkg.template/actions/workflows/linter.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/linter.yaml)
 [![Version bump](https://github.com/insightsengineering/r.pkg.template/actions/workflows/version-bump.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/version-bump.yaml)
 [![Version check](https://github.com/insightsengineering/r.pkg.template/actions/workflows/version.yaml/badge.svg)](https://github.com/insightsengineering/r.pkg.template/actions/workflows/version.yaml)
@@ -20,6 +21,7 @@
 [![Current Version](https://img.shields.io/github/r-package/v/insightsengineering/r.pkg.template/main?color=purple&label=Version@main)](https://github.com/insightsengineering/r.pkg.template/tree/main)
 [![Open Issues](https://img.shields.io/github/issues-raw/insightsengineering/r.pkg.template?color=red&label=Open%20Issues)](https://github.com/insightsengineering/r.pkg.template/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc)
 
+<!-- links -->
 [pre-commit]: https://pre-commit.com
 [pre-commit installation]: https://pre-commit.com/#installation
 [git hooks]: https://git-scm.com/book/en/v2/Customizing-Git-Git-Hooks