Skip to content

Commit

Permalink
fix: dependabot failures due to no secrets
Browse files Browse the repository at this point in the history
Dependabot does not get passed secrets. Just don't run the
actions that require secrets if they aren't passed in.
  • Loading branch information
elasticspoon committed May 14, 2024
1 parent 031c94d commit 81fea5a
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/combine_and_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@ jobs:
find artifacts -name "test_reports*.zip" -exec unzip -d test_reports {} \;
find test_reports -name "**/test_reports*.zip" -exec unzip -d test_reports {} \;
- name: Merge parallel runtime log parts
if: github.repository == github.event.pull_request.head.repo.full_name
if: secrets.ACCOUNT_NAME != ''
run: |
cat artifacts/**/parallel_runtime_rspec*.log > parallel_runtime.log
- name: Upload log file to Azure Blob Storage
if: github.repository == github.event.pull_request.head.repo.full_name
if: secrets.ACCOUNT_NAME != ''
env:
AZURE_STORAGE_KEY: ${{ secrets.STORAGE_ACCESS_KEY }}
AZURE_STORAGE_ACCOUNT: ${{ secrets.ACCOUNT_NAME }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/rspec.yml
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ jobs:
zip -r test_reports_${{ env.GROUPS_UNDERSCORE }}.zip tmp/reports
- name: Compress log
if: github.repository == github.event.pull_request.head.repo.full_name
if: secrets.ACCOUNT_NAME
run: |
mv tmp/parallel_runtime.log parallel_runtime_rspec_${{ env.GROUPS_UNDERSCORE }}.log
Expand All @@ -135,7 +135,7 @@ jobs:
path: test_reports_${{ env.GROUPS_UNDERSCORE }}.zip

- name: Upload file parallel tests runtime log
if: github.repository == github.event.pull_request.head.repo.full_name
if: secrets.ACCOUNT_NAME
uses: actions/upload-artifact@v4
with:
name: parallel_runtime_rspec_${{ env.GROUPS_UNDERSCORE }}.log
Expand Down

0 comments on commit 81fea5a

Please sign in to comment.