fix(ci): pin feature-ideation reusable workflow to SHA#105
fix(ci): pin feature-ideation reusable workflow to SHA#105
Conversation
Pin petry-projects/.github feature-ideation-reusable.yml from @v1 to commit SHA ae9709f4466dec60a5733c9e7487f69dcd004e05 to comply with the action-pinning policy. Also adds missing dry_run input to match the upstream standards template. Closes #90 Co-authored-by: don-petry <don-petry@users.noreply.github.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 51 minutes and 40 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
@don-petry PR #105 is ready for review. This fixes the compliance finding by pinning the |
|
There was a problem hiding this comment.
Pull request overview
Updates the repository’s Feature Ideation workflow stub to comply with the org action-pinning standard and align with the upstream workflow interface.
Changes:
- Pin the
feature-ideation-reusable.ymlreusable workflow reference from@v1to a specific commit SHA. - Add a
dry_runworkflow_dispatchinput and pass it through to the reusable workflow.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| include A, B, C. Key emerging trends in this space: X, Y, Z." | ||
| focus_area: ${{ inputs.focus_area || '' }} | ||
| research_depth: ${{ inputs.research_depth || 'standard' }} | ||
| dry_run: ${{ inputs.dry_run || false }} |
There was a problem hiding this comment.
workflow_dispatch inputs are provided as strings (even for type: boolean), so ${{ inputs.dry_run || false }} can evaluate to the string 'false' (truthy) and/or pass a non-boolean value into the reusable workflow. Coerce the dispatch input to a real boolean before passing it through (e.g., compare to 'true' or use fromJSON(...)) so the reusable receives an actual boolean and scheduled runs still default to false.
| dry_run: ${{ inputs.dry_run || false }} | |
| dry_run: ${{ fromJSON(inputs.dry_run || 'false') }} |
Automated review — APPROVEDRisk: LOW SummaryThis PR pins a reusable workflow reference from a floating FindingsInfo
CI statusAll CI checks pass: SonarCloud, CodeQL, AgentShield — zero new issues. Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with |



Summary
petry-projects/.github/.github/workflows/feature-ideation-reusable.ymlfrom@v1to commit SHAae9709f4466dec60a5733c9e7487f69dcd004e05to comply with the action-pinning policydry_runworkflow_dispatch input andwith:parameter to align with the upstream standards templateTest plan
Closes #90
Generated with Claude Code