TR updates, first round

lpozo · lpozo · commit 3ad1f71f5893 · 2022-11-14T19:54:48.000+01:00
diff --git a/python-argparse/calc.py b/python-argparse/calc.py
@@ -31,19 +31,19 @@ def div(a, b):
 }
 
 add_parser = subparsers.add_parser("add", help="add two numbers a and b")
-add_parser.add_argument(*arg_template)
+add_parser.add_argument(**arg_template)
 add_parser.set_defaults(func=add)
 
 sub_parser = subparsers.add_parser("sub", help="subtract two numbers a and b")
-sub_parser.add_argument(*arg_template)
+sub_parser.add_argument(**arg_template)
 sub_parser.set_defaults(func=sub)
 
 mul_parser = subparsers.add_parser("mul", help="multiply two numbers a and b")
-mul_parser.add_argument(*arg_template)
+mul_parser.add_argument(**arg_template)
 mul_parser.set_defaults(func=mul)
 
 div_parser = subparsers.add_parser("div", help="divide two numbers a and b")
-div_parser.add_argument(*arg_template)
+div_parser.add_argument(**arg_template)
 div_parser.set_defaults(func=div)
 
 args = global_parser.parse_args()
diff --git a/python-argparse/cooking.py b/python-argparse/cooking.py
@@ -2,7 +2,7 @@
 
 arg_parser = argparse.ArgumentParser()
 
-arg_parser.add_argument("--vegies", nargs="+")
+arg_parser.add_argument("--veggies", nargs="+")
 arg_parser.add_argument("--fruits", nargs="*")
 
 args = arg_parser.parse_args()
diff --git a/python-argparse/hello_cli/README.md b/python-argparse/hello_cli/README.md
@@ -0,0 +1,3 @@
+# Hello CLI
+
+My awesome Hello CLI application.
diff --git a/python-argparse/hello_cli/hello_cli/__init__.py b/python-argparse/hello_cli/hello_cli/__init__.py
@@ -0,0 +1 @@
+"""Hello CLI package."""
diff --git a/python-argparse/hello_cli/hello_cli/__main__.py b/python-argparse/hello_cli/hello_cli/__main__.py
@@ -0,0 +1 @@
+"""Entry-point for the hello_cli package."""
diff --git a/python-argparse/hello_cli/hello_cli/cli.py b/python-argparse/hello_cli/hello_cli/cli.py
@@ -0,0 +1 @@
+"""CLI-related code for the Hello CLI package."""
diff --git a/python-argparse/hello_cli/hello_cli/model.py b/python-argparse/hello_cli/hello_cli/model.py
@@ -0,0 +1 @@
+"""Back-end model for the hello_cli app."""
diff --git a/python-argparse/hello_cli/pyproject.toml b/python-argparse/hello_cli/pyproject.toml
@@ -0,0 +1,13 @@
+[build-system]
+requires = ["setuptools>=61.0.0", "wheel"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "hello_cli"
+version = "0.0.1"
+description = "My awesome Hello CLI application"
+readme = "README.md"
+authors = [{ name = "Real Python", email = "info@realpython.com" }]
+
+[project.scripts]
+hello_cli = "hello_cli.__main__:main"
diff --git a/python-argparse/hello_cli/requirements.txt b/python-argparse/hello_cli/requirements.txt
diff --git a/python-argparse/hello_cli/tests/__init__.py b/python-argparse/hello_cli/tests/__init__.py
diff --git a/python-argparse/hello_cli/tests/test_cli.py b/python-argparse/hello_cli/tests/test_cli.py
diff --git a/python-argparse/hello_cli/tests/test_model.py b/python-argparse/hello_cli/tests/test_model.py
diff --git a/python-argparse/ls_argv.py b/python-argparse/ls_argv.py
@@ -3,16 +3,16 @@
 
 if (args_count := len(sys.argv)) > 2:
     print(f"One argument expected, got {args_count - 1}")
-    sys.exit()
+    raise SystemExit(2)
 elif args_count < 2:
     print("You must specify the target directory")
-    sys.exit()
+    raise SystemExit(2)
 
 target_dir = Path(sys.argv[1])
 
 if not target_dir.is_dir():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 if __name__ == "__main__":
     for entry in target_dir.iterdir():
diff --git a/python-argparse/ls_v1.py b/python-argparse/ls_v1.py
@@ -1,5 +1,4 @@
 import argparse
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser()
@@ -12,7 +11,7 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 if __name__ == "__main__":
     for entry in target_dir.iterdir():
diff --git a/python-argparse/ls_v2.py b/python-argparse/ls_v2.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser()
@@ -15,17 +14,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v3.py b/python-argparse/ls_v3.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser(prog="ls")
@@ -15,17 +14,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v4.py b/python-argparse/ls_v4.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser(
@@ -19,17 +18,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v5.py b/python-argparse/ls_v5.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser(
@@ -21,17 +20,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v6.py b/python-argparse/ls_v6.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser(
@@ -22,17 +21,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v7.py b/python-argparse/ls_v7.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser(
@@ -22,17 +21,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v8.py b/python-argparse/ls_v8.py
@@ -1,6 +1,5 @@
 import argparse
 import datetime
-import sys
 from pathlib import Path
 
 parser = argparse.ArgumentParser(
@@ -32,17 +31,16 @@
 
 if not target_dir.exists():
     print("The target directory doesn't exist")
-    sys.exit()
+    raise SystemExit(1)
 
 
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 
diff --git a/python-argparse/ls_v9.py b/python-argparse/ls_v9.py
@@ -36,11 +36,10 @@
 def build_output(entry, long=False):
     if long:
         size = entry.stat().st_size
-        owner = entry.owner()
         date = datetime.date.fromtimestamp(entry.stat().st_mtime).strftime(
             "%b %d %m:%S"
         )
-        return f"{owner} {size:>6d} {date} {entry.name}"
+        return f"{size:>6d} {date} {entry.name}"
     return entry.name
 
 

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+# Hello CLI`
	`2`	`+`
	`3`	`+My awesome Hello CLI application.`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+"""Entry-point for the hello_cli package."""`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+"""CLI-related code for the Hello CLI package."""`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+"""Back-end model for the hello_cli app."""`