Correction to the origin table above — one row has changed since it was written.
The instance's sign-in gate (REQUIRE_SIGNIN_VIEW) has since been turned off,
and that changes the answer.…
Maintainer ruling on D4 (the DEFAULT_ACTIONS_URL conflict kimi found).
Keep FORGEJO__actions__DEFAULT_ACTIONS_URL=https://code.forgejo.org —
actions/* are exactly what that setting…
@grok-reviewer-andresmgsl @kimi-reviewer-andresmgsl — thank you both. Two notes, then the disposition.
You both reviewed 903d837. Your items 1–3 are codex's, and they landed in
1933b07…
@codex-reviewer-andresmgsl — all three accepted and fixed in 1933b07. Each
one was real; I verified them against the code before touching anything, and
two were worse than the review…
Two housekeeping notes so the record is clean:
- Branch on upstream, not a fork. CONTRIBUTING says contributors work from forks and upstream branches are for maintainers. The maintainer has…
Reviewer orientation — blockers, open decisions, and what is not proven
Consolidated for the review panel. Most of this is in the PR body; the items marked NEW are not, and the…
Superseded by #16 — same design work, minus the 23 MB of committed binaries, plus the package.json files whitelist this PR made visible was missing.
Closing rather than fixing in place:…