fix(curriculum): add actionable advanced OOP assertion messages - #142
fix(curriculum): add actionable advanced OOP assertion messages#142frobel0520 wants to merge 1 commit into
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: Pro Plus 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 |
|
Thanks for working on this! We ended up merging #139, which covers the same six checks, so I’m closing this PR as a duplicate. Really appreciate the effort here — and your work on the other curriculum batches too. |
Summary
checks/oop_advanced/oop_advanced7.pythroughoop_advanced12.py.Closes #108
Tests
python -m pytest tests/integration/test_solution_verify.py -q— 1 passedpython -m pytest -q— 209 passed, 6 failedWinError 1314) in doctor/manifest coverage; they are unrelated to this change.python -m pythonlings --root tests/fixtures/passing_curriculum verify— passed (passing1,passing2)Screenshots
Not applicable; this is a curriculum check-message change with no TUI changes.
Checklist
python -m pytest -q— completed; 209 passed and 6 unrelated Windows symlink-permission tests failed withWinError 1314.