ci: pin shared plugin setup to TLibs main - #25
Conversation
The setup-plugins action was pinned to 4f7a354, the head of TLibs PR #19, which is not on any TLibs branch. Pin it to 8ef41de on TLibs main. The action is unchanged; its installer catalogue now also lists GemInfusion. Drop -P'!deploy-live'. No POM defines a deploy-live profile, so the flag does nothing. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
|
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 configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 10 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe build and Maven release workflows use a new pinned shared plugin setup action. Their Maven version-setting, name-evaluation, and ChangesMaven Workflows
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The workflow changes do not alter repository-defined Maven profiles and are ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
A rabbit checks the workflow pin, Comment |
Summary
TF-Minecraft/TLibs/.github/actions/setup-pluginsto8ef41deon TLibsmain. The previous pin,4f7a354, is the head of TLibs PR build: compile against Paper API 1.21.10 #19 and is not on any branch (build: compile against Paper API 1.21.10 #19 was squash-merged as8f48e6f). The action itself is unchanged; since then the installer has only dropped its unused--assetsoption and added GemInfusion to its catalogue.-P'!deploy-live'from the Maven commands. No POM defines adeploy-liveprofile, so the flag does nothing.Verification
🤖 Generated with Claude Code
Summary by CodeRabbit