fix(ci): allow reusable tests workflow PR read scope#1979
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe pull request adds explicit Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Review rate limit: 3/5 reviews remaining, refill in 23 minutes and 4 seconds. Comment |
|



Summary (AI generated)
pull-requests: readto thebump_version.ymlreusable test workflow call.github/workflows/tests.ymlso GitHub can validate the nested workflowMotivation (AI generated)
The monorepo test workflow now declares
pull-requests: readfor its path filter job.bump_version.ymlcalls that workflow on pushes tomainanddevelopment, but its job-level permissions did not allow that scope, so GitHub rejected the workflow file before CI could start.Business Impact (AI generated)
This restores release automation on
mainanddevelopment. Without it, version bumping and follow-up release steps stop at workflow validation instead of running after merges.Test Plan (AI generated)
.github/workflows/bump_version.ymland.github/workflows/tests.ymllocally after the changeGenerated with AI
Summary by CodeRabbit