Skip to content

Commit 28bc12f

Browse files
chore(ci): add missing container-tag/container-suffix and fix stale standards doc (#138)
Add container-tag and container-suffix parameters to all reusable workflow jobs in ci.yml. Fix repository-standards.md: correct git hooks path, replace stale validation commands with vrg-docker-run -- vrg-validate, update tooling requirements, 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 f01fff0 commit 28bc12f

2 files changed

Lines changed: 11 additions & 12 deletions

File tree

.github/workflows/ci.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ jobs:
2626
language: shell
2727
versions: '["latest"]'
2828
container-suffix: base
29+
container-tag: 'latest'
2930

3031
security:
3132
uses: vergil-project/vergil-actions/.github/workflows/ci-security.yml@v2.0
@@ -34,6 +35,8 @@ jobs:
3435
run-standards: ${{ inputs.run-release != 'false' }}
3536
run-security: ${{ inputs.run-security != 'false' }}
3637
run-codeql: false
38+
container-suffix: base
39+
container-tag: 'latest'
3740
permissions:
3841
contents: read
3942
security-events: write
@@ -43,3 +46,5 @@ jobs:
4346
with:
4447
language: shell
4548
run-release: ${{ inputs.run-release != 'false' }}
49+
container-suffix: base
50+
container-tag: 'latest'

docs/repository-standards.md

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,12 @@
1414
for explicit approval to proceed on `develop`.
1515
- If approval is granted to work on `develop`, call it out in the
1616
response and proceed only for that user-approved scope.
17-
- Enable repository git hooks before committing: `git config core.hooksPath scripts/git-hooks`.
17+
- Enable repository git hooks before committing: `git config core.hooksPath .githooks`.
1818

1919
## Local validation
2020

2121
```bash
22-
# Validate the MQ environment is running and seeded correctly
23-
scripts/mq_verify.sh
24-
25-
# Lint documentation
26-
markdownlint . --ignore node_modules
22+
vrg-docker-run -- vrg-validate
2723
```
2824

2925
## Tooling requirement
@@ -32,7 +28,7 @@ Required for daily workflow:
3228

3329
- Docker and Docker Compose (for running the MQ containers)
3430
- `curl` (for REST API health checks and verification)
35-
- `markdownlint` (required for docs validation and PR pre-submission)
31+
- `vrg-docker-run` (runs validation inside dev container)
3632

3733
## Merge strategy override
3834

@@ -52,14 +48,13 @@ submission. Do not construct commit messages or PR bodies manually.
5248

5349
```bash
5450
vrg-commit \
55-
--type TYPE --message MESSAGE --agent AGENT \
51+
--type TYPE --message MESSAGE \
5652
[--scope SCOPE] [--body BODY]
5753
```
5854

5955
- `--type` (required): one of
6056
`feat|fix|docs|style|refactor|test|chore|ci|build`
6157
- `--message` (required): commit description
62-
- `--agent` (required): `claude` or `codex`
6358
- `--scope` (optional): conventional commit scope
6459
- `--body` (optional): detailed commit body
6560

@@ -72,17 +67,16 @@ The script resolves the correct `Co-Authored-By` identity from
7267
vrg-submit-pr \
7368
--issue NUMBER --summary TEXT \
7469
[--linkage KEYWORD] [--title TEXT] \
75-
[--notes TEXT] [--docs-only] [--dry-run]
70+
[--notes TEXT] [--dry-run]
7671
```
7772

7873
- `--issue` (required): GitHub issue number (just the number)
7974
- `--summary` (required): one-line PR summary
80-
- `--linkage` (optional, default: `Fixes`):
75+
- `--linkage` (optional, default: `Ref`):
8176
`Fixes|Closes|Resolves|Ref`
8277
- `--title` (optional): PR title (default: most recent commit
8378
subject)
8479
- `--notes` (optional): additional notes
85-
- `--docs-only` (optional): applies docs-only testing exception
8680
- `--dry-run` (optional): print generated PR without executing
8781

8882
The script detects the target branch and merge strategy

0 commit comments

Comments
 (0)