Closed matt-winkler closed 2 years ago
@joellabes Good feedback! I was wondering about the use of exclude here as well, so went ahead and added that one in + some love on integration tests.
I also moved the table_pattern
and exclude
parameters to the end of the macro args list in order to make a new minor version an option. Originally, was biasing towards matching the warehouse hierarchy (from lower to higher grain), but schema
being ahead of database
and required / non-default kind of defeats the purpose of that. Happy to move this around again if you prefer & update the Changelog accordingly.
This is a:
main
dev/
branchdev/
branchDescription & motivation
See here
Checklist