Skip to content

Add full-sweep-enabled label to split sweep tiers#1039

Open
n0madsky wants to merge 1 commit intoSemiAnalysisAI:mainfrom
n0madsky:967-full-sweep-enabled-label
Open

Add full-sweep-enabled label to split sweep tiers#1039
n0madsky wants to merge 1 commit intoSemiAnalysisAI:mainfrom
n0madsky:967-full-sweep-enabled-label

Conversation

@n0madsky
Copy link
Copy Markdown

Summary

  • Splits the single sweep-enabled CI label into two tiers: sweep-enabled (standard sweep with only the two configured concurrency endpoints per parallelism config) and full-sweep-enabled (full sweep with all intermediate concurrency points — the previous sweep-enabled behavior).
  • Push-to-main always gets the full sweep. Explicit --conc values remain authoritative and bypass the standard reduction.
  • Adds --full flag to test-config subcommand and forwards it through process_changelog.py and run-sweep.yml.
  • Adds 23 new tests covering standard/full sweep behavior, parser wiring, and edge cases (109 total passing).

Test plan

  • All 109 unit tests pass (pytest test_generate_sweep_configs.py)
  • No new ruff lint errors introduced (2 pre-existing warnings remain)
  • Verify sweep-enabled label triggers standard sweep (endpoint-only concurrency) on a test PR
  • Verify full-sweep-enabled label triggers full intermediate concurrency sweep on a test PR
  • Verify push-to-main triggers full sweep

Closes #967

🤖 Generated with Claude Code

Split the single `sweep-enabled` CI label into two tiers:
- `sweep-enabled`: standard sweep with only the two configured concurrency
  endpoints per parallelism config (new default for test-config)
- `full-sweep-enabled`: full sweep with all intermediate concurrency points
  (previous sweep-enabled behavior)

Push-to-main always gets full sweep. Explicit `--conc` values remain
authoritative and bypass the standard reduction.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@n0madsky n0madsky force-pushed the 967-full-sweep-enabled-label branch from ddde488 to fbfc566 Compare April 16, 2026 07:38
@n0madsky n0madsky marked this pull request as ready for review April 16, 2026 07:42
@n0madsky n0madsky requested a review from a team April 16, 2026 07:42
Copy link
Copy Markdown
Contributor

@claude claude bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

p0 dev velocity: full-sweep-enabled label

1 participant