fix: restore hashability of status enums that define custom __eq__#2470
Conversation
Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (6)
WalkthroughFive status enums now hash using their underlying values. New parameterized unit tests verify that enum members work in sets and dictionaries and remain hash-consistent with their integer values. ChangesStatus enum hashability
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
👋 Hi @exploreriii! Great work completing a Beginner issue! 🎉 Thanks for your contribution! 🚀 Here are some issues you might want to explore next:
🌟 Stay connected: Happy coding! 🚀 |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #2470 +/- ##
==========================================
+ Coverage 95.03% 95.07% +0.04%
==========================================
Files 164 164
Lines 10464 10474 +10
==========================================
+ Hits 9944 9958 +14
+ Misses 520 516 -4 🚀 New features to boost your workflow:
|
Fixes #2469