Fix Boolean CLI Arguments

gouline / dbt-metabase

dbt + Metabase integration

https://pypi.org/project/dbt-metabase/

MIT License

448 stars 64 forks source link

Closed erika-e closed 3 years ago

erika-e commented 3 years ago

See discussion on PR#29.

The current CLI arguments --mb_https and --sync can only be set to false from the command line by passing an empty string. This isn't intuitive.

Suggested changes:

Remove mb_https and propagate mb_http downstream to make programmatic interface consistent with CLI interface
Apply a similar change to --sync