BAM: Updated gempush.yml to run tests and linting on PRs (#567)

* BAM: Updated gempush.yml to run tests and linting on PRs

* BAM: Updated list of events to run workflow on

* BAM: Moving tests and linting to it's own workflow to run on master and PRs

* Trigger Build

* Adjust workflow name, add back tests to primary workflow

* Restrict permissions on linting workflow

* Trigger Build

* BAM: Attempting to call a workflow from a workflow

* BAM: Fix needs statement

* BAM: Adding  to tests-linting

* BAM: Removing requirement for secrets

Co-authored-by: Kevin Formsma <[email protected]>

Author: benniemosher

.github/workflows/gempush.yml

@@ -8,50 +8,12 @@ env:
   docker_org: stelligent
 
 jobs:
-  commit:
-    name: Commit
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false # Allows both ruby versions to run before failing
-      matrix:
-        ruby: ['2.5', '3.0']
-
-    steps:
-    - uses: actions/checkout@master
-    - name: Set up Ruby
-      uses: ruby/setup-ruby@v1
-      with:
-        ruby-version: ${{ matrix.ruby }}
-        bundler-cache: true
-    - name: Rubocop lint
-      run: bundle exec rubocop -D
-    - name: Rspec tests
-      run: bundle exec rspec --require spec_helper spec
-    - name: Duplicate rule check
-      run: bundle exec ./bin/cfn_nag_rules
-
-  acceptance:
-    name: Acceptance
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false # Allows both ruby versions to run before failing
-      matrix:
-        ruby: ['2.5', '3.0']
-
-    steps:
-    # is this the same commit hash we saw in commit?
-    - uses: actions/checkout@master
-    - name: Set up Ruby
-      uses: ruby/setup-ruby@v1
-      with:
-        ruby-version: ${{ matrix.ruby }}
-    - name: e2e
-      run: bash ./scripts/setup_and_run_end_to_end_tests.sh
-
+  tests-linting:
+    uses: stelligent/cfn_nag/.github/workflows/tests-linting.yml@master
   release:
     name: Release
     runs-on: ubuntu-latest
-    needs: [commit, acceptance]
+    needs: [tests-linting]
     steps:
       # is this the same commit hash we saw in commit?
       - uses: actions/checkout@v2

.github/workflows/tests-linting.yml

@@ -0,0 +1,55 @@
+name: test-lint
+
+on:
+  workflow_call:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+permissions:
+  contents: read
+
+env:
+  docker_org: stelligent
+
+jobs:
+  tests-linting:
+    name: Tests & Linting
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false # Allows both ruby versions to run before failing
+      matrix:
+        ruby: ['2.5', '3.0']
+
+    steps:
+    - uses: actions/checkout@master
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+        bundler-cache: true
+    - name: Rubocop lint
+      run: bundle exec rubocop -D
+    - name: Rspec tests
+      run: bundle exec rspec --require spec_helper spec
+    - name: Duplicate rule check
+      run: bundle exec ./bin/cfn_nag_rules
+
+  acceptance:
+    name: Acceptance
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false # Allows both ruby versions to run before failing
+      matrix:
+        ruby: ['2.5', '3.0']
+
+    steps:
+    # is this the same commit hash we saw in commit?
+    - uses: actions/checkout@master
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby }}
+    - name: e2e
+      run: bash ./scripts/setup_and_run_end_to_end_tests.sh