Skip to content

docs(settings): document that autoLabelEnabled only gates the base context label#5318

Merged
JSONbored merged 1 commit into
mainfrom
docs/auto-label-enabled-scope
Jul 12, 2026
Merged

docs(settings): document that autoLabelEnabled only gates the base context label#5318
JSONbored merged 1 commit into
mainfrom
docs/auto-label-enabled-scope

Conversation

@JSONbored

Copy link
Copy Markdown
Owner

Summary

`autoLabelEnabled` had no doc comment and reads like a master label switch. Its only real consumer gates exclusively the single base `gittensorLabel` context label — zero effect on TYPE/taxonomy labels (`typeLabelsEnabled`), moderation/blacklist labels, or review-state labels. This scoping was already documented on the sibling `typeLabelsEnabled` field and in the yml example, but not on `autoLabelEnabled` itself — and the dashboard toggle rendered as plain "Auto-label PRs" with no scoping hint, the most likely place a real maintainer would be misled.

Scope

Docs-only. `src/types.ts` (doc comment), `maintainer-settings.tsx` (dashboard hint).

Validation

  • `npm run typecheck` — clean
  • `npm run ui:typecheck` / `npm run ui:lint` — clean (0 errors)
  • `npm run docs:drift-check` — clean

Closes #5293

…ntext label

autoLabelEnabled had no doc comment and reads like a master label
switch. Its only real consumer gates exclusively the single base
gittensorLabel context label -- zero effect on TYPE/taxonomy labels
(typeLabelsEnabled), moderation/blacklist labels, or review-state
labels. This scoping was already documented on the sibling
typeLabelsEnabled field and in the yml example, but not on
autoLabelEnabled itself, and the dashboard toggle rendered as plain
"Auto-label PRs" with no scoping hint -- the most likely place a
real maintainer would be misled.

Added a doc comment mirroring the existing cross-reference style,
and a hint on the dashboard toggle.

Docs-only, no behavior change.

Refs #5293
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
gittensory-ui 9e39227 Commit Preview URL

Branch Preview URL
Jul 12 2026, 01:51 PM

@codecov

codecov Bot commented Jul 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.37%. Comparing base (dc6fa7a) to head (9e39227).
⚠️ Report is 8 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5318   +/-   ##
=======================================
  Coverage   94.37%   94.37%           
=======================================
  Files         474      474           
  Lines       40128    40128           
  Branches    14631    14631           
=======================================
  Hits        37871    37871           
  Misses       1582     1582           
  Partials      675      675           
Flag Coverage Δ
shard-1 46.28% <ø> (-0.28%) ⬇️
shard-2 34.74% <ø> (+0.06%) ⬆️
shard-3 32.17% <ø> (+0.07%) ⬆️
shard-4 32.09% <ø> (-0.15%) ⬇️
shard-5 33.38% <ø> (-0.11%) ⬇️
shard-6 45.14% <ø> (+0.30%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/types.ts 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@JSONbored JSONbored self-assigned this Jul 12, 2026
@JSONbored
JSONbored merged commit 6688156 into main Jul 12, 2026
17 checks passed
@JSONbored
JSONbored deleted the docs/auto-label-enabled-scope branch July 12, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document that autoLabelEnabled only gates the base gittensorLabel

1 participant