Repository navigation
evidence(OMN-12816): bind infra promotion receipt to dev PR - #2321
Conversation
|
Warning Review limit reached
More reviews will be available in 43 minutes and 49 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, 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 include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (10)
📝 WalkthroughWalkthroughA command.yaml evidence record was updated to reference PR ChangesPR Reference Update in Evidence Record
Possibly related PRs
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 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)
Comment |
482ed78 to
6762fd4
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@drift/dod_receipts/OMN-12816/dod-omn12816-main-promotion-infra/command.yaml`:
- Line 10: The receipt's contract_sha256 in command.yaml is incorrect; update
the contract_sha256 field (contract_sha256: "...") in command.yaml to exactly
match the SHA-256 value from contracts/OMN-12816.yaml
(sha256:b8999aa204bf6e03767d5a95b6b69513bf795569349745c0d90c45422ffb0511), or
alternatively replace the contract file contents so the SHA matches—ensure the
value in the command.yaml contract_sha256 key and the computed SHA of
contracts/OMN-12816.yaml are identical.
🪄 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
Run ID: 6aca315e-b29e-4d01-b91b-4b1fa8c8cdc0
📒 Files selected for processing (1)
drift/dod_receipts/OMN-12816/dod-omn12816-main-promotion-infra/command.yaml
6762fd4 to
d335802
Compare
Summary
Publishes the corrected OMN-12816 release evidence graph for the final infra promotion path.
Evidence Covered
omnibase_infra#1905: validdev -> mainpromotion PR for the OMN-12816 infra runtime change.omnibase_infra#1906:main -> devbackmerge PR needed to make the dev promotion mergeable.onex_change_control#2321: this OCC evidence correction PR, replacing obsolete evidence(OMN-12816): bind main promotion PR receipts #2320 self-binding.omnimarket#1108: already-merged market main promotion.Verification
uv run validate-yaml contracts/OMN-12816.yaml— PASSuv run pre-commit run yamlfmt --files contracts/OMN-12816.yaml drift/dod_receipts/OMN-12816/*/command.yaml— PASSgit diff --check— PASSNotes
Contract hash updated to
sha256:1bfd39aa9d234f80d37fa3ee3538baa68bbd33592da60e161a90967e24165907; all OMN-12816 receipts are restamped to that hash.