Skip to content

fix(CICD): revert workflow permissions from 'checks' to 'pull-requests'#145

Merged
lukeroantreeONS merged 9 commits intomainfrom
add_workflow_permissions
Mar 10, 2026
Merged

fix(CICD): revert workflow permissions from 'checks' to 'pull-requests'#145
lukeroantreeONS merged 9 commits intomainfrom
add_workflow_permissions

Conversation

@lukeroantreeONS
Copy link
Collaborator

📌 Pull Request Template

Please complete all sections

✨ Summary

📜 Changes Introduced

  • Feature implementation (feat:) / bug fix (fix:) / refactoring (chore:) / documentation (docs:) / testing (test:)
  • Updates to tests and/or documentation
  • Terraform changes (if applicable)

✅ Checklist

Please confirm you've completed these checks before requesting a review.

  • Code passes linting with Ruff
  • Security checks pass using Bandit
  • API and Unit tests are written and pass using pytest
  • Terraform files (if applicable) follow best practices and have been validated (terraform fmt & terraform validate)
  • DocStrings follow Google-style and are added as per Pylint recommendations
  • Documentation has been updated if needed

🔍 How to Test

@lukeroantreeONS lukeroantreeONS requested a review from a team as a code owner March 10, 2026 16:08
@lukeroantreeONS lukeroantreeONS changed the title Revert workflow permissions from 'checks' to 'pull-requests' Fix(CICD) revert workflow permissions from 'checks' to 'pull-requests' Mar 10, 2026
@lukeroantreeONS lukeroantreeONS changed the title Fix(CICD) revert workflow permissions from 'checks' to 'pull-requests' fix(CICD) revert workflow permissions from 'checks' to 'pull-requests' Mar 10, 2026
@lukeroantreeONS lukeroantreeONS changed the title fix(CICD) revert workflow permissions from 'checks' to 'pull-requests' fix(CICD): revert workflow permissions from 'checks' to 'pull-requests' Mar 10, 2026
@github-actions github-actions bot added the bug Something isn't working label Mar 10, 2026
Copy link
Collaborator

@rileyok-ons rileyok-ons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be good, lets go!

@lukeroantreeONS lukeroantreeONS merged commit 3c1164d into main Mar 10, 2026
5 of 8 checks passed
@lukeroantreeONS lukeroantreeONS deleted the add_workflow_permissions branch March 10, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants