Skip to content

bugfix/dont-run-terraform-for-dependabot-pr#202

Merged
chopkinsmade merged 1 commit into
mainfrom
bugfix/dont-run-terraform-for-dependabot-pr
Apr 16, 2026
Merged

bugfix/dont-run-terraform-for-dependabot-pr#202
chopkinsmade merged 1 commit into
mainfrom
bugfix/dont-run-terraform-for-dependabot-pr

Conversation

@chopkinsmade
Copy link
Copy Markdown
Contributor

@chopkinsmade chopkinsmade commented Apr 16, 2026

Description

PRs raised by dependabot fail due to their inability to access repo secrets. There is no way to make this work, so exclude the PR from running the terraform checks

Contributors

Let's acknowledge the people who contributed to the work.

Type of change

  • Refactoring (made code better without changing its behaviour)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How this has been tested

Please describe the tests that you ran to verify your changes.

If they are not automated tests please explain why and provide screenshots and/or instructions so they can reproduced.

Checklist

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Reviewer Checklist

  • I have reviewed the PR and ensured no secret values are present

Signed-off-by: DBT pre-commit check
@github-actions
Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

Scanned Files

None

@chopkinsmade chopkinsmade marked this pull request as ready for review April 16, 2026 15:15
@chopkinsmade chopkinsmade requested a review from a team as a code owner April 16, 2026 15:15
@chopkinsmade chopkinsmade merged commit 5f22668 into main Apr 16, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants