Skip to content

ci: map the #11680 codegen suites into e2e-scoped - #11779

Merged
proggeramlug merged 1 commit into
mainfrom
fix/e2e-scope-11680-suites
Oct 3, 2026
Merged

proggeramlug merged 1 commit into
mainfrom
fix/e2e-scope-11680-suites

Conversation

@proggeramlug

Copy link
Copy Markdown
Contributor

Map concat_site_agent_ownership and thread_agent_strings (added by #11680) into _CODEGEN_SUITES. Without them, every core PR's e2e-scoped job fails at scope selection ("in neither SOURCE_SUITE_MAP nor SUITE_EXCLUSIONS"), as on #11776.

python3 scripts/ci_e2e_scope.py --self-test passes, and both suites are now selected for a crates/perry-codegen/src change.

concat_site_agent_ownership and thread_agent_strings came in with #11680
but were in neither SOURCE_SUITE_MAP nor SUITE_EXCLUSIONS, so every core PR's
e2e-scoped job now fails at scope selection.
@coderabbitai

coderabbitai Bot commented Oct 3, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6d0efe86-99ae-48a4-935c-d74303d547a2
📥 Commits

Reviewing files that changed from the base of the PR and between 073b079 and 48c971f.

📒 Files selected for processing (1)
  • scripts/ci_e2e_scope.py
 ___________________________________________________________
< Patterns mean 'I have run out of language.' - Rich Hickey >
 -----------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@proggeramlug
proggeramlug merged commit cc678fe into main Oct 3, 2026
22 of 23 checks passed
@proggeramlug
proggeramlug deleted the fix/e2e-scope-11680-suites branch October 3, 2026 04:47
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.

1 participant