Making exposures output idempotent

gouline / dbt-metabase

dbt + Metabase integration

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

MIT License

442 stars 63 forks source link

Making exposures output idempotent #165

Closed ernesto-jimenez closed 5 months ago

ernesto-jimenez commented 1 year ago

Running the command twice changes the exposures file due to changes in the order of the outputs.

This results in noisy diffs which make it harder to follow what changed.

Proposal: When preparing the output, sort all arrays alphabetically.

gouline commented 1 year ago

I'm happy with the suggested approach. However, I'm not using the exposure functionality myself, that entire area of the codebase was contributed by the community. As a result, you (or somebody else interested) will have to work on that and contribute yourself.