Skip to content

Add GitHub Actions workflows, documentation, and Terraform review guidelines#3

Open
ColinDaglish wants to merge 13 commits into
review/initial-contentfrom
main
Open

Add GitHub Actions workflows, documentation, and Terraform review guidelines#3
ColinDaglish wants to merge 13 commits into
review/initial-contentfrom
main

Conversation

@ColinDaglish

Copy link
Copy Markdown
Contributor

Summary

Introduce GitHub Actions workflows for automated releases and Quarto publishing. Enhance documentation with Terraform review guidelines, environment descriptions, and a glossary.

Context

No related issue.

Changes

  • Added initial GitHub Actions workflows for release management and Quarto publishing.
  • Created comprehensive Terraform review guidelines and tools documentation.
  • Improved clarity in existing documentation and added new sections for environments and patterns.

Verification

Rendered locally with quarto render, no errors.

Risks and notes

Reviewers should pay attention to the new Terraform review guidelines and ensure they align with existing practices.


Sensitivity declaration

This repository is public. Tick all boxes before requesting review.

  • No credentials, API keys, tokens, or passwords
  • No personal data (names, emails, NI numbers, or other identifying information)
  • No internal hostnames, IP addresses, or project IDs not already publicly documented
  • No content marked OFFICIAL-SENSITIVE or above

ColinDaglish and others added 13 commits June 8, 2026 13:56
- Introduced a comprehensive pull request review checklist for Terraform.
- Added principles for effective Terraform code reviews.
- Created a guide on reading Terraform plan outputs.
- Provided best practices for writing review comments.
- Defined skills and expectations for reviewers at various experience levels.
- Documented essential review tools and their usage.
- Outlined specific aspects to review in Terraform code changes.
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.

1 participant