## Summary
Fixes the remaining documentation gap from issue #125.
PR #126 renamed `GITEA_URL` to `VCS_URL` in code, action YAML, and the env var table in README.md, but the CLI usage example still showed `--gitea-url`.
## Change
- Update README.md CLI example: `--gitea-url` → `--vcs-url`
## Completion Checklist
From PLAN-125.md:
- [x] `VCS_URL` is read first; `GITEA_URL` is fallback with deprecation warning (done in #126)
- [x] `--vcs-url` flag is primary; `--gitea-url` is deprecated alias with warning (done in #126)
- [x] Error message references `--vcs-url` not `--gitea-url` (done in #126)
- [x] `action.yml` passes `VCS_URL` to the binary (done in #126)
- [x] `ci.yml` passes `VCS_URL` to the binary (done in #126)
- [x] README updated in CLI example and env var table ← **this PR**
- [x] Integration tests use `INTEGRATION_VCS_URL` (done in #126)
- [x] `go test ./...` passes
- [x] `go vet ./...` passes
- [x] `go build ./cmd/review-bot` succeeds
Closes #125
No other stale --gitea-url references remain in README
## Self-review: PASS
No issues found — ready for human review.
**Change**: One-line doc fix — `--gitea-url` → `--vcs-url` in README CLI example.
**Verified**:
- `go build ./cmd/review-bot` ✅
- `go vet ./...` ✅
- `go test ./...` ✅
- No other stale `--gitea-url` references remain in README
Single-line documentation fix replacing the deprecated --gitea-url flag with --vcs-url in the CLI usage example. CI passed and the change is correct.
Recommendation
APPROVE — Approve. This is a minimal, correct documentation update that completes the migration from --gitea-url to --vcs-url in the README CLI example. CI passed, the change is consistent with the rest of the renamed env var table already present in the README, and no code is affected.
# Sonnet Review
## Summary
Single-line documentation fix replacing the deprecated `--gitea-url` flag with `--vcs-url` in the CLI usage example. CI passed and the change is correct.
## Recommendation
**APPROVE** — Approve. This is a minimal, correct documentation update that completes the migration from `--gitea-url` to `--vcs-url` in the README CLI example. CI passed, the change is consistent with the rest of the renamed env var table already present in the README, and no code is affected.
---
*Review by sonnet*
<!-- review-bot:sonnet -->
---
*Evaluated against d3bb83a1*
security-review-bot
requested review from security-review-bot 2026-05-14 06:51:25 +00:00
Trivial documentation update replacing --gitea-url with --vcs-url in the CLI example. No security implications and aligns with prior renaming work. CI passed.
Recommendation
APPROVE — Approve as is. The change correctly updates the CLI usage to reflect the new flag name and does not introduce any security risks. Consider scanning the rest of the README and related docs for any lingering references to --gitea-url to ensure complete consistency.
# Security Review
## Summary
Trivial documentation update replacing --gitea-url with --vcs-url in the CLI example. No security implications and aligns with prior renaming work. CI passed.
## Recommendation
**APPROVE** — Approve as is. The change correctly updates the CLI usage to reflect the new flag name and does not introduce any security risks. Consider scanning the rest of the README and related docs for any lingering references to --gitea-url to ensure complete consistency.
---
*Review by security*
<!-- review-bot:security -->
---
*Evaluated against d3bb83a1*
gpt-review-bot
approved these changes 2026-05-14 06:51:33 +00:00
The PR updates the README CLI example to use the new --vcs-url flag, aligning the documentation with prior code changes. The change is correct and consistent with the rest of the README.
Recommendation
APPROVE — This is a straightforward documentation fix replacing --gitea-url with --vcs-url in the CLI example. CI has passed and the change matches the established environment variable and flag mappings elsewhere in the README. Approve as-is.
# Gpt Review
## Summary
The PR updates the README CLI example to use the new --vcs-url flag, aligning the documentation with prior code changes. The change is correct and consistent with the rest of the README.
## Recommendation
**APPROVE** — This is a straightforward documentation fix replacing --gitea-url with --vcs-url in the CLI example. CI has passed and the change matches the established environment variable and flag mappings elsewhere in the README. Approve as-is.
---
*Review by gpt*
<!-- review-bot:gpt -->
---
*Evaluated against d3bb83a1*
rodin
merged commit d3bb83a10a into main2026-05-14 07:08:16 +00:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
sonnet-review-bot
security-review-bot
gpt-review-bot
aweiker
Summary
Fixes the remaining documentation gap from issue #125.
PR #126 renamed
GITEA_URLtoVCS_URLin code, action YAML, and the env var table in README.md, but the CLI usage example still showed--gitea-url.Change
--gitea-url→--vcs-urlCompletion Checklist
From PLAN-125.md:
VCS_URLis read first;GITEA_URLis fallback with deprecation warning (done in #126)--vcs-urlflag is primary;--gitea-urlis deprecated alias with warning (done in #126)--vcs-urlnot--gitea-url(done in #126)action.ymlpassesVCS_URLto the binary (done in #126)ci.ymlpassesVCS_URLto the binary (done in #126)INTEGRATION_VCS_URL(done in #126)go test ./...passesgo vet ./...passesgo build ./cmd/review-botsucceedsCloses #125
Self-review: PASS
No issues found — ready for human review.
Change: One-line doc fix —
--gitea-url→--vcs-urlin README CLI example.Verified:
go build ./cmd/review-bot✅go vet ./...✅go test ./...✅--gitea-urlreferences remain in READMESonnet Review
Summary
Single-line documentation fix replacing the deprecated
--gitea-urlflag with--vcs-urlin the CLI usage example. CI passed and the change is correct.Recommendation
APPROVE — Approve. This is a minimal, correct documentation update that completes the migration from
--gitea-urlto--vcs-urlin the README CLI example. CI passed, the change is consistent with the rest of the renamed env var table already present in the README, and no code is affected.Review by sonnet
Evaluated against
d3bb83a1Security Review
Summary
Trivial documentation update replacing --gitea-url with --vcs-url in the CLI example. No security implications and aligns with prior renaming work. CI passed.
Recommendation
APPROVE — Approve as is. The change correctly updates the CLI usage to reflect the new flag name and does not introduce any security risks. Consider scanning the rest of the README and related docs for any lingering references to --gitea-url to ensure complete consistency.
Review by security
Evaluated against
d3bb83a1Gpt Review
Summary
The PR updates the README CLI example to use the new --vcs-url flag, aligning the documentation with prior code changes. The change is correct and consistent with the rest of the README.
Recommendation
APPROVE — This is a straightforward documentation fix replacing --gitea-url with --vcs-url in the CLI example. CI has passed and the change matches the established environment variable and flag mappings elsewhere in the README. Approve as-is.
Review by gpt
Evaluated against
d3bb83a1