Skip to content

ci(playerbots): pin Core main with raid scope and rendered stops - #59

Merged
Mathih13 merged 1 commit into
mainfrom
ci/pin-core-raid-scope
Oct 3, 2026
Merged

Mathih13 merged 1 commit into
mainfrom
ci/pin-core-raid-scope

Conversation

@Mathih13

@Mathih13 Mathih13 commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

Packages CI pins Core 82e7cddc. Core main now has three merged changes that the next playerbots PRs need:

This PR moves all nine Core pins in four workflow files to the #606 merge commit dd287126. It changes no Package code. The raid suite (LyraCoreProject/LyraCore#534) and the runner stop (LyraCoreProject/LyraCore#570) follow on top of it.

Made by Claude Opus 5.5 in Claude Code.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated automated validation workflows to use a newer shared platform revision across core and player-bot checks.
    • The same revision is now used consistently across the affected workflows, keeping their test runs aligned.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@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.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: cc800c96-006d-4369-91c1-ab4ba17a38c9
📥 Commits

Reviewing files that changed from the base of the PR and between eda8fa2 and 5247337.

📒 Files selected for processing (4)
  • .github/workflows/core-tip.yml
  • .github/workflows/playerbots-capacity.yml
  • .github/workflows/playerbots-class-behavior.yml
  • .github/workflows/playerbots-target-claims.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Eight workflow checkout references changed from LyraCore commit 82e7cddc05fbfca382384fa568a9cd6e8315bfbe to dd287126ec8ee0ff58efc85fa745b309ed50f296.

Changes

LyraCore workflow checkouts

Layer / File(s) Summary
Update workflow checkout references
.github/workflows/core-tip.yml, .github/workflows/playerbots-capacity.yml, .github/workflows/playerbots-class-behavior.yml, .github/workflows/playerbots-target-claims.yml
Six jobs in core-tip.yml and the checkouts in three playerbots workflows now use the updated LyraCore commit.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Suggested reviewers: perpersen

Merge Risk: ⚪ Minimal · up to 52473

The CI workflows now use the updated LyraCore revision. No specific merge-blocking issue is evident from the supplied change context.

Architecture Summary

Architecture risk: 🔵 Low · up to 52473

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .github/workflows/core-tip.yml: The module job now checks out LyraCore at dd287126ec8ee0ff58efc85fa745b309ed50f296 instead of 82e7cddc05fbfca382384fa568a9cd6e8315bfbe.
  • observed — Modified behavior in .github/workflows/core-tip.yml: The datascripts job now checks out LyraCore at dd287126ec8ee0ff58efc85fa745b309ed50f296 instead of 82e7cddc05fbfca382384fa568a9cd6e8315bfbe.
  • observed — Modified behavior in .github/workflows/core-tip.yml: The playerbots-durable job now checks out LyraCore at dd287126ec8ee0ff58efc85fa745b309ed50f296 instead of 82e7cddc05fbfca382384fa568a9cd6e8315bfbe.
  • observed — Modified behavior in .github/workflows/core-tip.yml: The playerbots-transfer-durable job now checks out LyraCore at dd287126ec8ee0ff58efc85fa745b309ed50f296 instead of 82e7cddc05fbfca382384fa568a9cd6e8315bfbe.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the Core pin update and its raid-scope and rendered-stop changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 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.

@Mathih13
Mathih13 merged commit 89c2cc1 into main Oct 3, 2026
22 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.

1 participant