last-review

Author: delmorallopez

implement-cowsay/README.md

@@ -49,6 +49,8 @@ usage: cowsay [-h] [--animal {beavis,cheese,cow,daemon,dragon,fox,ghostbusters,k
 
 Make animals say things
 
+
+
 positional arguments:
   message               The message to say.
 

implement-cowsay/cow.py

@@ -1,28 +1,36 @@
-import cowsay
 import argparse
-import sys
-import os
+import cowsay
+
+def main():
+    # List of available animals (provided by the package)
+    animals = cowsay.char_names
+
+    parser = argparse.ArgumentParser(
+        prog="cowsay",
+        description="Make animals say things"
+    )
+
+    parser.add_argument(
+        "message",
+        nargs="+",
+        help="The message to say."
+    )
 
-# Dynamically determine the cows directory
-COWS_DIR = os.path.join(os.path.dirname(cowsay.__file__), "cows")
+    parser.add_argument(
+        "--animal",
+        choices=animals,
+        default="cow",
+        help="The animal to be saying things."
+    )
 
-# Set up argument parsing
-parser = argparse.ArgumentParser()
-parser.add_argument("--animal", required=True)
-parser.add_argument("message", nargs="+")
-args = parser.parse_args()
+    args = parser.parse_args()
 
-# Combine message parts into a single string
-message = " ".join(args.message)
+    message = " ".join(args.message)
 
-# Dynamically fetch available animals
-animals = [f[:-4] for f in os.listdir(COWS_DIR) if f.endswith(".cow")]
+    # Dynamically call the function for the chosen animal
+    animal_func = getattr(cowsay, args.animal)
+    print(animal_func(message))
 
-# Validate the provided animal
-if args.animal not in animals:
-    print("Error: '--animal' is not a valid cowsay animal.")
-    print("Available animals:", ", ".join(sorted(animals)))
-    sys.exit(1)
 
-# Just pass the animal name directly
-print(cowsay.cowsay(message, cow=args.animal))
+if __name__ == "__main__":
+    main()

implement-cowsay/requirements.txt

@@ -0,0 +1 @@
+cowsay