search

slidoapp / dbt-coverage

One-stop-shop for docs and test coverage of dbt projects.
MIT License
197 stars 25 forks source link

Adding contributing doc and pre-commit #73

Closed pgoslatara closed 7 months ago

pgoslatara commented 8 months ago

I really like this repo, however there is no guide for contributors. This PR adds a section to the README with a guide on how to setup a local development environment and how to run the integration tests locally. In addition, in order to maintain standard formatting and linting conventions, pre-commit is introduced with black and isort. The GitHub workflows now reflect this change, instead of calling just black, pre-commit is run, this will enable adding further hooks in the future (if desired).

pgoslatara commented 8 months ago

@sweco Is this good to merge or does it need another revision?

sweco commented 7 months ago

Thanks @pgoslatara, will release this. 😊