Skip to content

Commit

Permalink
fix: dbt workflows and readthedocs
Browse files Browse the repository at this point in the history
Bug fixes for CI around the new requirements versions and file
locations.
  • Loading branch information
bmtcril committed Oct 25, 2024
1 parent 5daa061 commit 6cb7a79
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 7 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/ci-dbt-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,14 @@ jobs:
- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: "3.11.x"
python-version: "3.12.x"
- name: Install dependencies
run: |
pip install tutor-contrib-aspects
pip install -r requirements/dbt.txt
cd dbt
pwd
ls -la
dbt deps
- name: Initialize Open edX
continue-on-error: false
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/ci-dbt-format.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,4 @@ jobs:
pip install -r requirements/dbt.txt
- name: Format SQL
run: |
cd dbt
sqlfmt models macros --check
sqlfmt dbt/models dbt/macros --check
4 changes: 2 additions & 2 deletions .readthedocs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,11 @@ sphinx:
build:
os: "ubuntu-22.04"
tools:
python: "3.8"
python: "3.12"
apt_packages:
- graphviz

# Optionally set the version of Python and requirements required to build your docs
python:
install:
- requirements: requirements/docs.txt
- requirements: requirements/docs.txt
6 changes: 6 additions & 0 deletions MANIFEST.in
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
include LICENSE
include README.rst
include requirements/base.in
include requirements/constraints.txt
recursive-include aspects *.html *.png *.gif *.js *.css *.jpg *.jpeg *.svg *.csv
recursive-include dbt *
5 changes: 3 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"""
Setup for xapi-db-load.
"""
from setuptools import find_packages, setup
from setuptools import setup

from aspects import __version__

Expand All @@ -11,7 +11,8 @@
setup(
name="openedx-aspects",
version=__version__,
packages=find_packages(),
packages=["dbt", "aspects"],
package_data={'dbt': ['**/*']},
include_package_data=True,
entry_points="""
[console_scripts]
Expand Down

0 comments on commit 6cb7a79

Please sign in to comment.