Conversation
…tten against The adapter's behaviour is decided by this fork, so resolving the name from Hex tested a different event store than the one it ships with. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
…ying Every advisory in the tree had a published fix, and the decimal one needed a newer ecto first before its major release could be taken. Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
PR SummaryMedium Risk Overview
Reviewed by Cursor Bugbot for commit aa40f2f. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. WalkthroughThe optional ChangesEventstore Dependency
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The optional Eventstore dependency now points to the intended pinned fork. No actionable merge-blocking issue is established, although the fork’s manifest remains unverified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings
🧪 Generate unit tests (beta)
🛠️ Fix failing CI checks 💡
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. A rabbit checks the tag with care, Comment |
|
Superseded by #112, same change with an allowed commit type. |
:eventstoreby name from Hex pulled a different event store than the one this adapter is written for, so the adapter was compiled and tested against code that does not ship with it. The subscription fixes it depends on were never exercised.Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.