Skip to content

add some tests and release machinery#18

Merged
jaxxstorm merged 5 commits into
mainfrom
ci
Apr 17, 2026
Merged

add some tests and release machinery#18
jaxxstorm merged 5 commits into
mainfrom
ci

Conversation

@jaxxstorm

Copy link
Copy Markdown
Collaborator

Signed-off-by: Lee Briggs lee@leebriggs.co.uk

Signed-off-by: Lee Briggs <lee@leebriggs.co.uk>
Copilot AI review requested due to automatic review settings April 15, 2026 17:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds Terraform test coverage and GitHub Actions workflows to validate the module and publish GitHub releases on tags.

Changes:

  • Add terraform test assertions for rendered cloud-init output and relay_server_port validation.
  • Add CI workflow running pre-commit, terraform-docs diff check, and terraform test.
  • Add release workflow to generate GitHub releases from tags.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
tests/rendered.tftest.hcl New Terraform tests asserting rendered output and variable validation behavior.
.github/workflows/ci.yml New CI pipeline: pre-commit validation, docs diff enforcement, and terraform unit tests.
.github/workflows/release.yml New tag-triggered workflow to create/update GitHub releases with generated notes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/rendered.tftest.hcl Outdated
Comment thread tests/rendered.tftest.hcl Outdated
Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
jaxxstorm and others added 4 commits April 15, 2026 11:41
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Lee Briggs <lee@leebriggs.co.uk>
Signed-off-by: Lee Briggs <lee@leebriggs.co.uk>
Signed-off-by: Lee Briggs <lee@leebriggs.co.uk>

@zbuchheit zbuchheit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@jaxxstorm jaxxstorm merged commit 342bd51 into main Apr 17, 2026
3 checks passed
@jaxxstorm jaxxstorm deleted the ci branch April 17, 2026 02:05
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.

3 participants