Skip to content

Commit

Permalink
Fix Python build (#1638)
Browse files Browse the repository at this point in the history
* test

* update sqlfluff

* update poetry

* disable new sqlfluff rule

* remove compile from workflow
  • Loading branch information
ravenac95 authored Jun 14, 2024
1 parent ade9548 commit 9e74e68
Show file tree
Hide file tree
Showing 8 changed files with 19 additions and 19 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-default.yml
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ jobs:
uses: packetcoders/action-setup-cache-python-poetry@main
with:
python-version: 3.12
poetry-version: 1.7.1
poetry-version: 1.8.2

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/deploy-dbt-docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:
uses: packetcoders/action-setup-cache-python-poetry@main
with:
python-version: 3.12
poetry-version: 1.7.1
poetry-version: 1.8.2

- name: Login to google
uses: "google-github-actions/auth@v2"
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/deploy-hasura.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
uses: packetcoders/action-setup-cache-python-poetry@main
with:
python-version: 3.12
poetry-version: 1.7.1
poetry-version: 1.8.2

- name: Login to google
uses: 'google-github-actions/auth@v2'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-deploy/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ runs:
uses: packetcoders/action-setup-cache-python-poetry@main
with:
python-version: 3.12
poetry-version: 1.7.1
poetry-version: 1.8.2

- name: Install poetry deps in the pr-clone
shell: bash
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/warehouse-run-data-pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ jobs:
uses: packetcoders/action-setup-cache-python-poetry@main
with:
python-version: 3.12
poetry-version: 1.7.1
poetry-version: 1.8.2

- name: Run poetry install
run: |
Expand Down
1 change: 1 addition & 0 deletions .sqlfluff
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ templater = dbt
runaway_limit = 10
max_line_length = 80
indent_unit = space
exclude_rules = AL09

[sqlfluff:indentation]
tab_space_size = 4
Expand Down
21 changes: 10 additions & 11 deletions poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ sqlalchemy = "^2.0.25"
google-cloud-storage = "^2.14.0"
python-dotenv = "^1.0.1"
dbt-bigquery = "^1.7.0"
dbt-core = "^1.8.2"
textual = "^0.52.1"
google-cloud-service-management = "^1.8.3"
google-cloud-resource-manager = "^1.12.3"
Expand All @@ -40,7 +41,6 @@ polars = "^0.20.23"
requests = "^2.31.0"
dagster-postgres = "^0.23.6"
pytest = "^8.2.1"
dbt-core = "^1.8.2"


[tool.poetry.scripts]
Expand All @@ -49,8 +49,8 @@ oso_lets_go = 'oso_lets_go.wizard:run'

[tool.poetry.group.dev.dependencies]
ipython = "^8.21.0"
sqlfluff = "^2.3.5"
sqlfluff-templater-dbt = "^2.3.5"
sqlfluff = "^3.0.7"
sqlfluff-templater-dbt = "^3.0.7"
shandy-sqlfmt = { extras = ["jinjafmt"], version = "^0.21.1" }
dagster-webserver = "^1.7.2"

Expand Down

0 comments on commit 9e74e68

Please sign in to comment.