Skip to content

Commit

Permalink
👷 [maykinmedia/open-api-framework#92] Make sure docs are built in CI
Browse files Browse the repository at this point in the history
  • Loading branch information
stevenbal committed Dec 19, 2024
1 parent 4715811 commit 668b14f
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
12 changes: 9 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ on:

env:
IMAGE_NAME: maykinmedia/open-klant
DJANGO_SETTINGS_MODULE: openklant.conf.ci
DB_PASSWORD: ''
DB_USER: postgres
# DJANGO_SETTINGS_MODULE: openklant.conf.ci
# DB_PASSWORD: ''
# DB_USER: postgres

jobs:
# determine changed files to decide if certain jobs can be skipped or not
Expand Down Expand Up @@ -79,7 +79,10 @@ jobs:
python src/manage.py collectstatic --noinput --link
coverage run src/manage.py test src
env:
DJANGO_SETTINGS_MODULE: objects.conf.ci
SECRET_KEY: dummy
DB_USER: postgres
DB_PASSWORD: ''

- name: Publish coverage report
uses: codecov/codecov-action@v3
Expand All @@ -105,6 +108,8 @@ jobs:
echo "Please update the environment documentation by running \`bin/generate_envvar_docs.sh\`"
exit 1
fi
env:
DJANGO_SETTINGS_MODULE: objects.conf.ci

store-reusable-workflow-vars:
name: create values which can be passed through a reusable workflow
Expand All @@ -123,6 +128,7 @@ jobs:
- store-reusable-workflow-vars
with:
main-branch: 'master'
run-docs: true
python-version: '3.11'
docker-image-name: ${{ needs.store-reusable-workflow-vars.outputs.image-name }}

Expand Down
2 changes: 1 addition & 1 deletion docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@
# The theme to use for HTML and HTML Help pages. See the documentation for
# a list of builtin themes.
#
html_logo = "logo.svg"
# html_logo = "logo.svg"
html_theme = "sphinx_rtd_theme"

# Add any paths that contain custom static files (such as style sheets) here,
Expand Down

0 comments on commit 668b14f

Please sign in to comment.