Skip to content

Commit 3fed192

Browse files
committed
docs(config): note settings.* as a legacy alias for 11 gate/AI-review fields
Part of loopover#6444 (Batch C): confirms gate.* is already the canonical, exclusively-documented form for reviewCheckMode, linkedIssueGateMode, duplicatePrGateMode, qualityGateMode, qualityGateMinScore, selfAuthoredLinkedIssueGateMode, and the five aiReview* fields across every .loopover.yml example and self-hosting doc. Adds the still-supported settings.* legacy-alias spellings to the two exhaustive reference files (.loopover.yml.example, loopover.full.yml) and a short note in tuning.mdx, since those files previously didn't mention this alias at all for these fields.
1 parent 32d0ef8 commit 3fed192

3 files changed

Lines changed: 46 additions & 0 deletions

File tree

.loopover.yml.example

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -730,6 +730,26 @@ review:
730730
# Everything a maintainer can toggle in the dashboard can be set here as code.
731731
# All values shown are the safe defaults; delete any line to inherit it.
732732
settings:
733+
# The 11 fields below each have a friendlier `gate.<name>` alias documented in section 2 above
734+
# (`gate.checkMode`/`gate.enabled`, `gate.linkedIssue`, `gate.duplicates`, `gate.readiness.mode`/
735+
# `gate.readiness.minScore`, `gate.selfAuthoredLinkedIssue`, `gate.aiReview.mode`/`.byok`/`.provider`/
736+
# `.model`/`.allAuthors`). `gate.*` is the RECOMMENDED spelling going forward and wins whenever both
737+
# forms are set for the same field (see PRECEDENCE at the top of this file). The generic names below
738+
# remain fully supported (no functional deprecation, still DB-backed) and are shown here, commented
739+
# out, only so this reference stays discoverable for a config written before the `gate:` alias
740+
# existed. Prefer `gate.*` above for anything new.
741+
# reviewCheckMode: visible # alias of gate.checkMode / gate.enabled
742+
# linkedIssueGateMode: advisory # alias of gate.linkedIssue
743+
# duplicatePrGateMode: block # alias of gate.duplicates
744+
# qualityGateMode: advisory # alias of gate.readiness.mode
745+
# qualityGateMinScore: null # alias of gate.readiness.minScore
746+
# selfAuthoredLinkedIssueGateMode: advisory # alias of gate.selfAuthoredLinkedIssue
747+
# aiReviewMode: off # alias of gate.aiReview.mode
748+
# aiReviewByok: false # alias of gate.aiReview.byok
749+
# aiReviewProvider: null # alias of gate.aiReview.provider
750+
# aiReviewModel: null # alias of gate.aiReview.model
751+
# aiReviewAllAuthors: false # alias of gate.aiReview.allAuthors
752+
733753
# Who receives the public PR comment.
734754
# off | detected_contributors_only | all_prs. Default: detected_contributors_only.
735755
commentMode: detected_contributors_only

apps/loopover-ui/content/docs/tuning.mdx

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,12 @@ gate evaluation, comments, labels, audit, or autonomous merge/close on or off
205205
dimension modes below directly, and set `gate.checkMode` explicitly instead of
206206
the ambiguous `gate.enabled`. The main dimensions:
207207

208+
Several of these also have a generic `settings.<field>` spelling (for example
209+
`settings.linkedIssueGateMode`, `settings.qualityGateMode`,
210+
`settings.aiReviewMode`) — that spelling remains fully supported for backward
211+
compatibility, but the `gate.*` form shown below is the recommended one going
212+
forward and wins whenever both are set for the same field.
213+
208214
- `gate.pack` — the policy pack: `gittensor` (default; registry-aware,
209215
tracks confirmed-Gittensor-contributor status for scoring) or `oss-anti-slop`
210216
(runs the deterministic rules against any author on any repo, with no

config/examples/loopover.full.yml

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -744,6 +744,26 @@ review:
744744
# Everything a maintainer can toggle in the dashboard can be set here as code.
745745
# All values shown are the safe defaults; delete any line to inherit it.
746746
settings:
747+
# The 11 fields below each have a friendlier `gate.<name>` alias documented in section 2 above
748+
# (`gate.checkMode`/`gate.enabled`, `gate.linkedIssue`, `gate.duplicates`, `gate.readiness.mode`/
749+
# `gate.readiness.minScore`, `gate.selfAuthoredLinkedIssue`, `gate.aiReview.mode`/`.byok`/`.provider`/
750+
# `.model`/`.allAuthors`). `gate.*` is the RECOMMENDED spelling going forward and wins whenever both
751+
# forms are set for the same field (see PRECEDENCE at the top of this file). The generic names below
752+
# remain fully supported (no functional deprecation, still DB-backed) and are shown here, commented
753+
# out, only so this reference stays discoverable for a config written before the `gate:` alias
754+
# existed. Prefer `gate.*` above for anything new.
755+
# reviewCheckMode: visible # alias of gate.checkMode / gate.enabled
756+
# linkedIssueGateMode: advisory # alias of gate.linkedIssue
757+
# duplicatePrGateMode: block # alias of gate.duplicates
758+
# qualityGateMode: advisory # alias of gate.readiness.mode
759+
# qualityGateMinScore: null # alias of gate.readiness.minScore
760+
# selfAuthoredLinkedIssueGateMode: advisory # alias of gate.selfAuthoredLinkedIssue
761+
# aiReviewMode: off # alias of gate.aiReview.mode
762+
# aiReviewByok: false # alias of gate.aiReview.byok
763+
# aiReviewProvider: null # alias of gate.aiReview.provider
764+
# aiReviewModel: null # alias of gate.aiReview.model
765+
# aiReviewAllAuthors: false # alias of gate.aiReview.allAuthors
766+
747767
# Who receives the public PR comment.
748768
# off | detected_contributors_only | all_prs. Default: detected_contributors_only.
749769
commentMode: detected_contributors_only

0 commit comments

Comments
 (0)