feat: add CI/CD workflows for infrastructure validation#53
Closed
feat: add CI/CD workflows for infrastructure validation#53
Conversation
Add 5 GitHub Actions workflows to validate infrastructure changes: - shell-quality.yml: ShellCheck and shfmt for bash scripts - commit-format.yml: Conventional commit validation - verify-session-handoff.yml: Session handoff documentation check - pr-title-check.yml: PR title format validation - protect-master.yml: Block direct pushes to master All workflows use reusable workflows from maxrantil/.github@master. This closes the critical gap of having zero CI/CD validation for 607 lines of infrastructure code (Terraform, Ansible, bash scripts). Resolves: #48 (or latest issue number) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add 2 additional GitHub Actions workflows: - block-ai-attribution.yml: Blocks PRs with AI tool attribution - pre-commit-validation.yml: Runs pre-commit hooks in CI This brings vm-infra to full parity with project-templates standard workflow set (7 total workflows). All workflows use reusable workflows from maxrantil/.github@master. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add workflow to validate PR body/description for AI attribution, completing the coverage alongside commit and issue checks. Uses new pr-body-ai-attribution-check-reusable workflow from maxrantil/.github (feature branch for testing). Once the .github PR is merged, this will be updated to @master.
4 tasks
Now that the reusable workflow is merged to master in .github repo, update the reference from feature branch to @master.
Owner
Author
|
Closing PR as CI/CD implementation approach has changed. The workflows will be reintroduced through a different strategy. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds 5 GitHub Actions workflows to validate infrastructure changes before merge. This closes the critical gap of having zero CI/CD for 607 lines of infrastructure code.
Workflows Added
1. Shell Quality (
shell-quality.yml)provision-vm.sh,destroy-vm.sh, test scripts**.shortests/**changes2. Commit Format (
commit-format.yml)feat:,fix:,docs:,refactor:3. Session Handoff (
verify-session-handoff.yml)4. PR Title Check (
pr-title-check.yml)5. Master Protection (
protect-master.yml)Technical Details
All workflows use reusable workflows from
maxrantil/.github@master:shell-quality-reusable.ymlconventional-commit-check-reusable.ymlsession-handoff-check-reusable.ymlpr-title-check-reusable.ymlprotect-master-reusable.ymlTest Plan
.github/workflows/@masterbranch referenceExpected CI Time
Impact
Before: Infrastructure changes merged without validation
After: 5 automated checks prevent infrastructure breakage
Resolves
Closes #48 (or latest CI/CD issue number)
Part of Full Agents Audit implementation (Week 1 - CRITICAL priority)
Documentation
Will update README with CI/CD section in follow-up PR.
🤖 Generated with Claude Code