Skip to content

Commit 0472d81

Browse files
wphillipmooreclaude
andcommitted
chore(ci): add missing container-tag/container-suffix and fix stale standards doc
Add container-tag and container-suffix parameters to all reusable workflow jobs (audit, quality, security, test, version) in ci.yml. Fix repository-standards.md: correct git hooks path from sibling vergil-tooling checkout to .githooks, replace bundle exec rake with vrg-docker-run -- vrg-validate, remove stale --agent and --docs-only flags, fix --linkage default from Fixes to Ref. Addresses vergil-project/vergil-tooling#1015, vergil-project/vergil-tooling#1013 Co-Authored-By: Claude <noreply@anthropic.com>
1 parent bb1c3af commit 0472d81

2 files changed

Lines changed: 15 additions & 7 deletions

File tree

.github/workflows/ci.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@ jobs:
3131
with:
3232
language: ruby
3333
versions: ${{ inputs.ruby-versions || '["3.2", "3.3", "3.4"]' }}
34+
container-suffix: ruby
35+
container-tag: 'latest'
3436

3537
integration-tests:
3638
name: "test / integration / ${{ matrix.ruby-version }}"
@@ -72,13 +74,17 @@ jobs:
7274
with:
7375
language: ruby
7476
versions: ${{ inputs.ruby-versions || '["3.2", "3.3", "3.4"]' }}
77+
container-suffix: ruby
78+
container-tag: 'latest'
7579

7680
security:
7781
uses: vergil-project/vergil-actions/.github/workflows/ci-security.yml@v2.0
7882
with:
7983
language: ruby
8084
run-standards: ${{ inputs.run-release != 'false' }}
8185
run-security: ${{ inputs.run-security != 'false' }}
86+
container-suffix: ruby
87+
container-tag: 'latest'
8288
permissions:
8389
contents: read
8490
security-events: write
@@ -88,9 +94,13 @@ jobs:
8894
with:
8995
language: ruby
9096
versions: ${{ inputs.ruby-versions || '["3.2", "3.3", "3.4"]' }}
97+
container-suffix: ruby
98+
container-tag: 'latest'
9199

92100
version:
93101
uses: vergil-project/vergil-actions/.github/workflows/ci-version-bump.yml@v2.0
94102
with:
95103
language: ruby
96104
run-release: ${{ inputs.run-release != 'false' }}
105+
container-suffix: ruby
106+
container-tag: 'latest'

docs/repository-standards.md

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,14 @@
1414
- Before modifying any files, check the current branch with `git status -sb`.
1515
- If on `develop`, create a short-lived `feature/*` branch or ask for explicit approval to proceed on `develop`.
1616
- If approval is granted to work on `develop`, call it out in the response and proceed only for that user-approved scope.
17-
- Enable repository git hooks before committing: `git config core.hooksPath ../vergil-tooling/scripts/lib/git-hooks`.
17+
- Enable repository git hooks before committing: `git config core.hooksPath .githooks`.
1818

1919
## Local validation
2020

2121
Canonical local validation command:
2222

2323
```bash
24-
bundle exec rake
24+
vrg-docker-run -- vrg-validate
2525
```
2626

2727
## Tooling requirement
@@ -53,14 +53,13 @@ and PR submission. Do not construct commit messages or PR bodies manually.
5353

5454
```bash
5555
vrg-commit \
56-
--type TYPE --message MESSAGE --agent AGENT \
56+
--type TYPE --message MESSAGE \
5757
[--scope SCOPE] [--body BODY]
5858
```
5959

6060
- `--type` (required): one of
6161
`feat|fix|docs|style|refactor|test|chore|ci|build`
6262
- `--message` (required): commit description
63-
- `--agent` (required): `claude` or `codex`
6463
- `--scope` (optional): conventional commit scope
6564
- `--body` (optional): detailed commit body
6665

@@ -73,17 +72,16 @@ The script resolves the correct `Co-Authored-By` identity from
7372
vrg-submit-pr \
7473
--issue NUMBER --summary TEXT \
7574
[--linkage KEYWORD] [--title TEXT] \
76-
[--notes TEXT] [--docs-only] [--dry-run]
75+
[--notes TEXT] [--dry-run]
7776
```
7877

7978
- `--issue` (required): GitHub issue number (just the number)
8079
- `--summary` (required): one-line PR summary
81-
- `--linkage` (optional, default: `Fixes`):
80+
- `--linkage` (optional, default: `Ref`):
8281
`Fixes|Closes|Resolves|Ref`
8382
- `--title` (optional): PR title (default: most recent commit
8483
subject)
8584
- `--notes` (optional): additional notes
86-
- `--docs-only` (optional): applies docs-only testing exception
8785
- `--dry-run` (optional): print generated PR without executing
8886

8987
The script detects the target branch and merge strategy

0 commit comments

Comments
 (0)