Manchester | 25-SDC-Nov | Rahwa Haile | Sprint 4 | Implement cowsay in Python #269
Conversation
RahwaZeslusHaile
added
the
Needs Review
|
Your PR couldn't be matched to an assignment in this module. Please check its title is in the correct format, and that you only have one PR per assignment. If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
github-actions
bot
removed
the
Needs Review
|
Your PR couldn't be matched to an assignment in this module. Please check its title is in the correct format, and that you only have one PR per assignment. If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed). If this PR needs reviewed, please add the 'Needs Review' label to this PR after you have resolved the issues listed above. |
RahwaZeslusHaile
changed the title
RahwaZeslusHaile
added
the
Needs Review
LonMcGregor
added
the
Review in progress
|
It looks like you may be missing some committs, can you check? |
LonMcGregor
added
Reviewed
2 participants
Manchester | 25-SDC-Nov | Rahwa Haile | Sprint 4 | Implement cowsay in Python
Learners, PR Template
Self checklist
Changelist
Mimics the basic behavior of the cowsay command line tool in Python.
Supports a message input and optional --animal selection.
Shows an error for unsupported animals, similar to the official CLI.
Questions
The implementation currently mimics the official cowsay CLI by showing an error when an unsupported animal is provided, using a hardcoded list of animals. Is this approach acceptable, or would you recommend a different method for handling unsupported animals? Thank you!