fix: remove redundant heimdallr job from release workflow (#11)

Author: benniemosher

.github/workflows/release.yml

@@ -7,20 +7,13 @@ on:
     branches:
       - main
 
+permissions: read-all
+
 jobs:
   release:
-    uses: "Mosher-Labs/.github/.github/workflows/release.yml@main"
-    permissions:
-      contents: write
-
-  heimdallr:
-    needs: release
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-    uses: "Mosher-Labs/.github/.github/workflows/heimdallr.yml@main"
+    uses: "Mosher-Labs/.github/.github/workflows/release.yml@v0.10.3"
+    secrets: inherit
     permissions:
       contents: write
       issues: write
       pull-requests: write
-    secrets: inherit
-    with:
-      slack_message: "The project <https://github.com/${{ github.repository }}|${{ github.repository }}> just released version <https://github.com/${{ github.repository }}/releases/tag/${{ needs.release.outputs.next_version }}|${{ needs.release.outputs.next_version }}>."

.pre-commit-config.yaml

@@ -36,7 +36,7 @@ repos:
     hooks:
       - id: actionlint-docker
   - repo: https://github.com/bridgecrewio/checkov
-    rev: 3.2.493
+    rev: 3.2.494
     hooks:
       - id: checkov_container
   - repo: https://github.com/compilerla/conventional-pre-commit

CLAUDE.md

@@ -420,6 +420,29 @@ If you need to rebuild the cluster:
 - **k3s token:** Generated during server install, not stored in Git
 - **Kubeconfig:** Retrieved from server, stored locally (not in Git)
 
+## Git Workflow
+
+1. **Create feature branch:** `git checkout -b feature/description`
+1. **Make changes** to playbooks, roles, or documentation
+1. **ALWAYS run pre-commit BEFORE committing:** `pre-commit run --all-files`
+   - Fix ALL errors (especially ansible-lint, yamllint, and markdown)
+   - Do NOT commit with `--no-verify` unless absolutely necessary
+1. **Commit with conventional format:** `git commit -m "type: description"`
+1. **Push and create PR:** `gh pr create --title "feat: description"`
+1. **Test changes:** If your changes reference shared workflows that were also updated,
+   temporarily change the reference from `@main` to `@your-branch` to test, verify
+   the PR passes, then change back to `@main` before merging
+1. **Merge to main:** Ansible playbooks are ready to run
+
+**Commit Format:** Conventional Commits (enforced by pre-commit hook)
+
+- `feat:` - New feature or role
+- `fix:` - Bug fix
+- `docs:` - Documentation changes
+- `chore:` - Maintenance
+- `refactor:` - Code refactoring
+- `test:` - Temporary test changes (like branch references)
+
 ## TODO / Future Improvements
 
 - [ ] Setup and configure molecule tests