Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use pyproject.toml for CLI parameter default #21

Open
jacobeatsspam opened this issue Jul 21, 2024 · 0 comments
Open

Use pyproject.toml for CLI parameter default #21

jacobeatsspam opened this issue Jul 21, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@jacobeatsspam
Copy link

jacobeatsspam commented Jul 21, 2024

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.

@jacobeatsspam jacobeatsspam changed the title pyproject.toml doesn't seem to work Use pyproject.toml for CLI parameter default Jul 22, 2024
@tedivm tedivm added the bug Something isn't working label Jul 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants