Skip to content

Commit

Permalink
removed dbt deps from pre-commit checks and updated set up docs
Browse files Browse the repository at this point in the history
  • Loading branch information
britt-allen committed Sep 3, 2024
1 parent f16dd6e commit c072ae4
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 9 deletions.
2 changes: 2 additions & 0 deletions {{project_name}}/.github/workflows/pre-commit.yml.jinja
Original file line number Diff line number Diff line change
Expand Up @@ -56,3 +56,5 @@ jobs:
run: |
poetry install
- uses: pre-commit/[email protected]
- name: Install dbt deps
run: poetry run dbt deps --project-dir transform
9 changes: 0 additions & 9 deletions {{project_name}}/.pre-commit-config.yaml.jinja
Original file line number Diff line number Diff line change
Expand Up @@ -20,15 +20,6 @@ repos:
hooks:
- id: yamllint
args: []
- repo: local
hooks:
- name: Dbt deps
id: dbt-deps
language: system
entry: poetry run dbt deps --project-dir transform
always_run: false
pass_filenames: false
types: [sql]
# Note: for SQLFluff we don't use the default pre-commit hook for a few reasons:
# 1. dbt cloud looks for config in the dbt project directory, so we put it in
# transform/, however sqlfluff requires the templater config to be in the
Expand Down
12 changes: 12 additions & 0 deletions {{project_name}}/docs/setup.md.jinja
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,18 @@ poetry install --with dev

Any time the dependencies change, you can re-run the above command to update them.

### 3. Install dbt dependencies

dbt comes with some core libraries, but others can be added on.
This command installs those extra libraries that are needed for this repo.
It will have to be re-run each time the repo starts relying on a new dbt library.

To install dbt dependencies, open a terminal and enter:

```bash
poetry run dbt deps --project-dir transform
```

## Configure Snowflake

In order to use Snowflake (as well as the terraform validators for the Snowflake configuration)
Expand Down

0 comments on commit c072ae4

Please sign in to comment.