tedivm / paracelsus

Visualize SQLAlchemy Databases using Mermaid or Dot Diagrams.
https://pypi.org/project/paracelsus/
MIT License
91 stars 9 forks source link

Use pyproject.toml for CLI parameter default #21

Open jacobeatsspam opened 2 months ago

jacobeatsspam commented 2 months ago

No matter what I try, using pyproject.toml seems to have no effect when trying to use inject, but using the same value in base works when specified in the command line.

If I can suggest a fix:

    base_class_path: Annotated[
        str,
        typer.Argument(help="The SQLAlchemy base class used by the database to graph."),
    ] = PYPROJECT_SETTINGS.get("base_class_path"),

This appears to be a general problem with the CLI functions. I think it would be worth setting this defaults across the board and removing the redundant calls to get_pyproject_settings.