-
Notifications
You must be signed in to change notification settings - Fork 15
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Feature: Update packages and Evidence to USQL (#23)
* Update packages after long gap * Update packages, notably Evidence for USQL * Delete misplaced issue source * Expand README * Make tables incremental * Remove markdownlint file * Resolve package conflicts hopefully * Update workflow file versions * Use uv in CI * Update pre-commit * Sort out sqlfluff more * uv needs venv in CI * Think venv needs to reactivate after reqs * Trying to get it to recognize duckdb in CI * Okay EL is working now dbt is failing * Yea for some reason you have to reactivate venv each step * Hack for uv needing a venv * Moar Hack for uv needing a venv * Moar Hack for uv needing a venv Pt 2 * Revert attempt at uv for now they are working on it * Try including config in workflows dir * Yea didn't think that would work * Try throwing flags in case it's missing config file? * Lets get verbose output from sqlfluff * Pin workflow to 3.11.5 idk * Lint for non-incremental compilation * CI should work now i think * dbt+sqlfluff need to be in same runner * Clean up Evidence CI step * Add environment to allow md to connect * Use single duckdb source called 'quack' * Try using secret as env var for Evidence CI build * Remove env var use only for deploys
- Loading branch information
1 parent
aabcb7d
commit 82698f6
Showing
31 changed files
with
5,693 additions
and
3,103 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
on: | ||
pull_request: | ||
branches: | ||
- main | ||
jobs: | ||
ci: | ||
name: CI Check | ||
runs-on: macos-latest | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4.1.1 | ||
- name: Setup Python | ||
uses: actions/setup-python@v5.0.0 | ||
with: | ||
python-version: "3.11.x" | ||
- name: Setup Node | ||
uses: actions/setup-node@v4.0.2 | ||
with: | ||
node-version: 20.x | ||
- name: Install uv | ||
run: python3 -m pip install uv | ||
- name: Install Python requirements | ||
run: uv pip install -r requirements.txt | ||
- name: Install Node requirements | ||
run: npm install --prefix ./reports | ||
- name: Run EL | ||
run: python3 el.py -lc | ||
- name: Run T | ||
run: dbt deps && dbt build | ||
- name: Lint SQL | ||
run: sqlfluff lint models --format github-annotation-native | ||
- name: Build Evidence | ||
env: | ||
EVIDENCE_DATABASE: "duckdb" | ||
EVIDENCE_DUCKDB_FILENAME: "octocatalog.db" | ||
run: npm run sources && npm run build --prefix ./reports |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
20.* |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,28 +1,29 @@ | ||
repos: | ||
- repo: https://github.com/pre-commit/pre-commit-hooks | ||
rev: v2.3.0 | ||
rev: v4.5.0 | ||
hooks: | ||
- id: check-yaml | ||
exclude: reports/evidence.plugins.yaml | ||
- id: end-of-file-fixer | ||
- id: trailing-whitespace | ||
- id: requirements-txt-fixer | ||
- repo: https://github.com/charliermarsh/ruff-pre-commit | ||
rev: v0.0.291 | ||
rev: v0.2.2 | ||
hooks: | ||
- id: ruff | ||
args: [--fix, --exit-non-zero-on-fix] | ||
- repo: https://github.com/PyCQA/isort | ||
rev: 5.12.0 | ||
rev: 5.13.2 | ||
hooks: | ||
- id: isort | ||
- repo: https://github.com/psf/black | ||
rev: 23.9.1 | ||
rev: 24.2.0 | ||
hooks: | ||
- id: black | ||
- repo: https://github.com/sqlfluff/sqlfluff | ||
rev: 2.3.2 | ||
rev: 2.3.5 | ||
hooks: | ||
- id: sqlfluff-fix | ||
args: ["./models/"] | ||
additional_dependencies: | ||
["dbt-metricflow[duckdb]~=0.3.0", "sqlfluff-templater-dbt"] | ||
["dbt-duckdb~=1.7.1", "sqlfluff-templater-dbt~=2.3.5"] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,6 @@ | ||
target/ | ||
dbt_packages/ | ||
macros/ | ||
.venv | ||
reports/ | ||
generate_fixtures.sql |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.