Refactor pipelines to get R version only once for downstream workflows (

#2199)

* Test no R version

* Consolidate check templates into common pipeline

* change admiralci ref

* remove pharmaverseadam push

---------

Co-authored-by: Daphné Grasselly <[email protected]>

.github/workflows/common.yml

@@ -33,31 +33,48 @@ concurrency:
   group: admiral-${{ github.event.pull_request.number || github.ref }}
   cancel-in-progress: true
 
+env:
+  R_VERSION: "4.1"
+
 jobs:
+  get_r_version:
+    name: Get R version
+    runs-on: ubuntu-latest
+    outputs:
+      r-version: ${{ steps.get_r_version.outputs.R_VERSION }}
+    steps:
+      - name: Get R Version for Downstream Container Jobs
+        id: get_r_version
+        run: echo "R_VERSION=$R_VERSION" >> $GITHUB_OUTPUT
+        shell: bash
   style:
     name: Code Style
     uses: pharmaverse/admiralci/.github/workflows/style.yml@main
     if: github.event_name == 'pull_request'
+    needs: get_r_version
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
   spellcheck:
     name: Spelling
     uses: pharmaverse/admiralci/.github/workflows/spellcheck.yml@main
     if: github.event_name == 'pull_request'
+    needs: get_r_version
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
   readme:
     name: Render README
     uses: pharmaverse/admiralci/.github/workflows/readme-render.yml@main
     if: github.event_name == 'push'
+    needs: get_r_version
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
   validation:
     name: Validation
     uses: pharmaverse/admiralci/.github/workflows/r-pkg-validation.yml@main
     if: github.event_name == 'release'
+    needs: get_r_version
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
   check:
     name: Check
     uses: pharmaverse/admiralci/.github/workflows/r-cmd-check.yml@main
@@ -68,6 +85,7 @@ jobs:
     name: Documentation
     uses: pharmaverse/admiralci/.github/workflows/pkgdown.yml@main
     if: github.event_name == 'push' || startsWith(github.ref, 'refs/tags/v')
+    needs: get_r_version
     with:
       r-version: "4.3"
       skip-multiversion-docs: true
@@ -76,9 +94,10 @@ jobs:
   linter:
     name: Lint
     uses: pharmaverse/admiralci/.github/workflows/lintr.yml@main
+    needs: get_r_version
     if: github.event_name == 'pull_request'
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
   links:
     name: Links
     uses: pharmaverse/admiralci/.github/workflows/links.yml@main
@@ -88,8 +107,9 @@ jobs:
     uses: pharmaverse/admiralci/.github/workflows/code-coverage.yml@main
     if: >
       github.event_name != 'release'
+    needs: get_r_version
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
       # Whether to skip code coverage badge creation
       # Setting to 'false' will require you to create
       # an orphan branch called 'badges' in your repository
@@ -98,5 +118,15 @@ jobs:
     name: Man Pages
     uses: pharmaverse/admiralci/.github/workflows/man-pages.yml@main
     if: github.event_name == 'pull_request'
+    needs: get_r_version
+    with:
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
+  templates:
+    name: Check Templates
+    uses: pharmaverse/admiralci/.github/workflows/check-templates.yml@main
+    secrets: inherit
+    if: github.event_name == 'pull_request' || github.event_name == 'workflow_dispatch'
+    needs: get_r_version
     with:
-      r-version: "4.1"
+      r-version: "${{ needs.get_r_version.outputs.r-version }}"
+      push-templates-data: false