Merge pull request #22 from schubergphilis/fix-gh-actin

update github action
schubergphilis · Dec 28, 2022 · bf89d62 · bf89d62
2 parents 4f349ce + ce8f320
commit bf89d62
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/.github/workflows/terraform.yml b/.github/workflows/terraform.yml
@@ -1,5 +1,5 @@
 ---
-name: Terraform
+name: "Terraform"
 
 on:
   pull_request:
@@ -47,6 +47,7 @@ jobs:
         uses: actions/checkout@v2
         with:
           ref: ${{ github.event.pull_request.head.ref }}
+        continue-on-error: true # added this to prevent a PR from a remote fork failing the workflow
       - name: Update module usage docs and push any changes back to PR branch
         uses: Dirrk/terraform-docs@v1.0.8
         with:
@@ -56,14 +57,14 @@ jobs:
           tf_docs_output_file: README.md
           tf_docs_output_method: inject
           tf_docs_find_dir: .
+        continue-on-error: true # added this to prevent a PR from a remote fork failing the workflow
 
   tfsec:
-    name: tfsec
     runs-on: ubuntu-latest
     steps:
       - name: Check out code
-        uses: actions/checkout@v2
-      - name: Terraform security scan
-        uses: aquasecurity/tfsec-pr-commenter-action@v1.1.0
+        uses: actions/checkout@master
+      - name: Terraform pr commenter
+        uses: aquasecurity/tfsec-pr-commenter-action@v1.2.0
         with:
           github_token: ${{ github.token }}