West Midlands | 26 March SDC | Iswat Bello | Sprint 4 | implement cowsay#495
Open
Iswanna wants to merge 2 commits into
Open
West Midlands | 26 March SDC | Iswat Bello | Sprint 4 | implement cowsay#495Iswanna wants to merge 2 commits into
Iswanna wants to merge 2 commits into
Conversation
- Add .venv to ignore Python virtual environments - Add *.class to ignore Java compiled class files
- Create cow.py with argparse for command-line argument parsing - Support dynamic animal selection with --animal flag (defaults to "cow") - Accept message as positional arguments and join them - Use getattr() to dynamically call animal functions from cowsay library - Fetch available animals from cowsay.CHARS instead of hardcoding - Add requirements.txt with cowsay dependency
Iswanna
added
📅 Sprint 4
behnamkvl
added
the
Review in progress
|
This implementation looks good to me and it works as expected. |
behnamkvl
added
Reviewed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Learners, PR Template
Self checklist
Changelist
Pull Request Summary
In this Pull request, I Implemented a command-line cowsay tool in Python that allows users to make animals say custom messages.
Changes
cow.py: New Python script using
argparsefor command-line argument parsingcowsaylibrary--animalflag to select which animal speaks (defaults to "cow")getattr()to dynamically call the appropriate animal functionrequirements.txt: Added
cowsaydependencyFeatures
--helpflag--animalspecifiedUsage Examples
python3 cow.py Grass, delicious. python3 cow.py --animal turtle Fish are cool! python3 cow.py --help