Skip to content

chore(repo): specify missing workflow permissions in CICD#141

Open
lukeroantreeONS wants to merge 3 commits intomainfrom
add_workflow_permissions
Open

chore(repo): specify missing workflow permissions in CICD#141
lukeroantreeONS wants to merge 3 commits intomainfrom
add_workflow_permissions

Conversation

@lukeroantreeONS
Copy link
Collaborator

@lukeroantreeONS lukeroantreeONS commented Mar 5, 2026

📌 specify missing workflow permissions in ci.yml

✨ Summary

Code Scanning alerted that workflow permissions are not explicitly set in our CICD pipeline, so they default to the permissions of the repo owners which is overly permissive.

This PR limits them to 'write', which is needed for the conventional commits task.

📜 Changes Introduced

  • chore(repo): limit CI.yml workflow permissions to 'write'

✅ 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

Check the Lint & Test workflow output that runs on opening this PR

@lukeroantreeONS lukeroantreeONS requested a review from a team as a code owner March 5, 2026 20:40
@github-actions github-actions bot added the chore label Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant