settings: Move command-line parser setup out of Settings class

We want the Settings-related command-line options to be co-located with the Settings class, as the two likely need to be changed together, when we add/modify settings to FawltyDeps. Still, they don't necessarily need to be part of the same _class_. This makes the Settings class somewhat smaller. Suggested-by: Maria Knorps <[email protected]>
tweag · Feb 23, 2023 · da5877b · da5877b
1 parent dd36630
commit da5877b
Show file tree

Hide file tree

Showing 2 changed files with 190 additions and 190 deletions.
diff --git a/fawltydeps/main.py b/fawltydeps/main.py
@@ -27,7 +27,7 @@
 from fawltydeps import extract_imports
 from fawltydeps.check import compare_imports_to_dependencies
 from fawltydeps.extract_declared_dependencies import extract_declared_dependencies
-from fawltydeps.settings import Action, OutputFormat, Settings
+from fawltydeps.settings import Action, OutputFormat, Settings, setup_cmdline_parser
 from fawltydeps.types import (
     DeclaredDependency,
     ParsedImport,
@@ -156,7 +156,7 @@ def print_human_readable(self, out: TextIO, details: bool = True) -> None:
 
 def main() -> int:
     """Command-line entry point."""
-    parser, option_group = Settings.setup_cmdline_parser(description=__doc__)
+    parser, option_group = setup_cmdline_parser(description=__doc__)
     option_group.add_argument(
         "-V",
         "--version",