feat: adds proof ingestion workflow#42
Conversation
Signed-off-by: Amit Singh <singhamitch@outlook.com>
Signed-off-by: Amit Singh <singhamitch@outlook.com>
Signed-off-by: Amit Singh <singhamitch@outlook.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ 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: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR separates proof handling from the existing document-posting pipeline by introducing two new workflows: ChangesProof Pipeline Separation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
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)
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 |
Signed-off-by: Amit Singh <singhamitch@outlook.com>
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/workflows/fetch_ingest_proofs.yml:
- Around line 24-31: The workflow uses floating action tags
(actions/checkout@v4, actions/setup-python@v6) and leaves checkout credentials
persisted; update the checkout and setup-python steps to pin to specific commit
SHAs instead of version tags and add persist-credentials: false to the
actions/checkout step to avoid exposing tokens. Locate the Checkout step (uses:
actions/checkout) and replace the tag with the canonical commit SHA for the
desired release, add the persist-credentials: false key under its with block,
and likewise replace actions/setup-python@v6 with its exact commit SHA to ensure
immutability.
In @.github/workflows/post_proofs_on_merge.yml:
- Around line 9-12: The job "post" currently relies on default workflow
permissions; add explicit least-privilege permissions for this job by setting
permissions.contents: read (since it only needs repo checkout/read access) under
the "post" job definition so the workflow doesn't inherit broader-than-needed
rights.
- Around line 13-19: The workflow uses mutable tags for actions and omits
disabling persisted credentials: replace the loose versions actions/checkout@v4
and actions/setup-python@v6 with their corresponding full commit SHAs and add
persist-credentials: false to the actions/checkout step; update the checkout
step (actions/checkout) to include the persist-credentials: false key and pin
both actions (actions/checkout and actions/setup-python) to exact commit SHAs to
ensure immutability.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: bbc8e6da-ad30-4423-b6a9-62c3b6d7c43f
📒 Files selected for processing (13)
.github/workflows/fetch_ingest_proofs.yml.github/workflows/post_on_merge.yml.github/workflows/post_proofs_on_merge.yml.github/workflows/refresh_scores.ymlproofs/ai-backlash/idahonews.yamlproofs/bbc/fifa-italy-iran/supports/theweek.yamlproofs/china-zero-tarrif/gov-cn.yamlproofs/eu-sa-trade-deal/policy-trade-europa.yamlproofs/fifa-wc-caution/amnestyusa.yamlproofs/india-gdp/gov-in.yamlproofs/moore-threads-share/investing.yamlproofs/nyt/pentagon-google-ai/supports/war-gov.yamlproofs/us-tarrifs/economic-times.yaml
💤 Files with no reviewable changes (2)
- proofs/nyt/pentagon-google-ai/supports/war-gov.yaml
- proofs/bbc/fifa-italy-iran/supports/theweek.yaml
There was a problem hiding this comment.
3 issues found across 13 files
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
Summary by CodeRabbit