Skip to content

chore(wip-limit): raise Code-review nudge threshold 8 → 30#59

Merged
aptracebloc merged 1 commit into
mainfrom
chore/wip-limit-30
Jun 17, 2026
Merged

chore(wip-limit): raise Code-review nudge threshold 8 → 30#59
aptracebloc merged 1 commit into
mainfrom
chore/wip-limit-30

Conversation

@saadqbal

@saadqbal saadqbal commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

What

Bumps the limit default in the reusable wip-limit-check.yml workflow from 8 → 30.

Why

The workflow posts a soft "👋 Heads-up — Code review queue is at N / LIMIT" nudge when the kanban Code review column is over the WIP limit. At 8 the threshold is too low for current throughput — e.g. tracebloc-client#390 was nudged at 19 / 8. Raising the bucket to 30 means the nudge only fires above 30 in review.

Scope

Every caller (tracebloc-client, backend, client-runtime, client, cli, docs, frontend-app, tracebloc-website, tracebloc-py-package, data-ingestors) uses uses: tracebloc/.github/.github/workflows/wip-limit-check.yml@main with secrets: inherit and no limit override — so this one default change applies org-wide.

Base branch note

Targeting main (not develop): the reusable workflow is pinned @main, and develop is currently 8 commits behind main with nothing unique — so main is the live source and the only base that makes the change take effect with a clean one-line diff.

Closes #58.

🤖 Generated with Claude Code


Note

Low Risk
Single default value change in a non-blocking CI nudge workflow with no logic or secret handling changes.

Overview
Raises the default limit input on the reusable wip-limit-check.yml workflow from 8 to 30, so the kanban “Code review” queue nudge only posts when depth is above 30.

Callers that rely on the default (no limit override) pick up the new threshold org-wide; workflow behavior and the nudge comment logic are unchanged.

Reviewed by Cursor Bugbot for commit 32462b5. Bugbot is set up for automated code reviews on this repo. Configure here.

The reusable wip-limit-check workflow nudges new PRs when the kanban
"Code review" column is over the WIP limit. The default of 8 is too low
for current throughput (e.g. tracebloc-client#390 nudged at 19/8). Bump
the bucket to 30. All caller repos inherit this default; none override.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@saadqbal saadqbal self-assigned this Jun 17, 2026
@LukasWodka

Copy link
Copy Markdown
Contributor

👋 Heads-up — Code review queue is at 17 / 8

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

  • .github#57 — fix(fr-gate): pass items at or beyond the required stage · author: @aptracebloc · no reviewer assigned
  • backend#813 — test(#805): contract test for tokenizer fingerprint storage (Task 2) · author: @shujaatTracebloc · reviewer: @aptracebloc
  • backend#815 — chore(deps): bump cryptography from 47.0.0 to 48.0.1 · author: @dependabot · no reviewer assigned
  • cli#78 — fix(dataset rm): delete staging files from a uid-65532 pod, not jobs-manager (#259) · author: @LukasWodka · no reviewer assigned
  • cli#79 — chore(schema): re-sync vendored ingest.v1.json from data-ingestors master · author: @LukasWodka · no reviewer assigned
  • client#261 — feat(installer): fail fast when HOST_DATA_DIR is on a network filesystem · author: @saadqbal · no reviewer assigned
  • client#262 — feat(installer,chart): place datasets on a network mount while MySQL stays local · author: @saadqbal · no reviewer assigned
  • client-runtime#108 — fix(authz): match ingest table prefixes at a segment boundary (close cross-tenant straddle) · author: @LukasWodka · no reviewer assigned
  • client-runtime#114 — fix(jobs): cap training Job backoffLimit to stop crashloops starving the cluster · author: @saadqbal · no reviewer assigned
  • client-runtime#115 — feat(ingestion): run the ingestion pod as the host uid for datasets on NFS · author: @saadqbal · no reviewer assigned

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

@aptracebloc aptracebloc merged commit 31ce01c into main Jun 17, 2026
5 checks passed
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.

Raise Code-review WIP-limit nudge threshold from 8 to 30

3 participants