Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Covers every bare assert listed in abhiksark#86 - 9 asserts across checks/functional/functional1..6 - so a failing check tells a beginner what was expected and what the code produced instead. Assert predicates, order and the success prints are unchanged. Assertions that guard a value shape (map/filter results, sorted lists) also name the likely cause, e.g. "wrap map(...) in list(...)".
b5a438e to
a87b66e
Compare
mercael91
changed the title
|
Pushed an update: Add actionable assertion messages to all batch-1 functional checks. Ready for re-review when you have a moment. |
1 participant
Closes #86.
Adds beginner-facing messages to every bare assertion listed in the issue (9 asserts across 6 files):
checks/functional/functional1.py-square(0)checks/functional/functional2.py-absolute(0),absolute(-100)checks/functional/functional3.py-isinstance(doubled, list)checks/functional/functional4.py-isinstance(evens, list), the even-numbers check overevenschecks/functional/functional5.py- first and last item ofby_lengthchecks/functional/functional6.py-apply_twice(lambda x: x + 1, 0)Each message states what was expected and what the code produced instead; the ones guarding a value's shape (map/filter results, sorted list) also name the likely cause, e.g. "wrap map(...) in list(...)". Predicates, assertion order and the success prints are unchanged.
Verified locally by running each check through the repository runner (
pythonlings.core.runner.run) against the reference answers insolutions/_answers.py: all six pass, and wrong answers fail with the new messages.