ci: pin feature-ideation-reusable.yml to SHA (action pinning compliance)#203
ci: pin feature-ideation-reusable.yml to SHA (action pinning compliance)#203don-petry wants to merge 3 commits into
Conversation
Pin petry-projects/.github/.github/workflows/feature-ideation-reusable.yml from @v1 to @ee22b427cbce9ecadcf2b436acb57c3adf0cb63d # v1 to satisfy the action-pinning policy (ci-standards.md#action-pinning-policy). Also syncs template updates from standards/workflows/feature-ideation.yml: - Add dry_run workflow_dispatch input - Add actions: read permission (feed checkpoint support) - Add sources_file comment guidance Closes #159 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 56 minutes and 17 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 is ready for review and merge. This is the CODEOWNERS review request. Summary of changes:
Note: PR #183 (the previous attempt) is still open — you may want to close it after merging this one. |
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Pins the reusable “feature ideation” workflow to a commit SHA for GitHub Action pinning compliance, and syncs recent template improvements (new dry_run input, required permissions, and source-list guidance).
Changes:
- Pin reusable workflow reference from
@v1to a full commit SHA. - Add
dry_runworkflow_dispatch input and pass-through to the reusable workflow. - Expand permissions/docs: add
actions: readand document repo-localsources_fileguidance.
| # sources_file: 'docs/feature-ideation-sources.md' | ||
| focus_area: ${{ inputs.focus_area || '' }} | ||
| research_depth: ${{ inputs.research_depth || 'standard' }} | ||
| dry_run: ${{ inputs.dry_run || false }} |
There was a problem hiding this comment.
inputs.dry_run is already a boolean with a default (false) in workflow_dispatch. Using || false is redundant and can be slightly confusing about intended typing/coercion. Prefer passing it through directly (e.g., dry_run: ${{ inputs.dry_run }}) to keep the expression simpler and preserve the boolean value without relying on || semantics.
| dry_run: ${{ inputs.dry_run || false }} | |
| dry_run: ${{ inputs.dry_run }} |
|
Summary
petry-projects/.github/.github/workflows/feature-ideation-reusable.ymlfrom@v1to@ee22b427cbce9ecadcf2b436acb57c3adf0cb63d # v1standards/workflows/feature-ideation.yml: addsdry_runinput,actions: readpermission, andsources_filecomment guidanceDetails
The SHA was looked up via
gh api repos/petry-projects/.github/git/refs/tags/v1— thev1lightweight tag currently points toee22b427cbce9ecadcf2b436acb57c3adf0cb63d. The standards template uses an older SHA (ae9709f4...) so I used the current authoritative value from the tag.Closes #159
Generated with Claude Code