Skip to content

Commit

Permalink
checkpoint
Browse files Browse the repository at this point in the history
  • Loading branch information
aucampia committed Mar 17, 2023
1 parent 1a2cd04 commit d3835e3
Show file tree
Hide file tree
Showing 7 changed files with 169 additions and 227 deletions.
5 changes: 0 additions & 5 deletions .github/workflows/validate.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ jobs:
os: ubuntu-latest
extensive-tests: true
TOXENV_SUFFIX: "-docs"
WITH_KROKI_CONTAINER: true
- python-version: "3.7"
os: ubuntu-latest
extensive-tests: true
Expand All @@ -45,7 +44,6 @@ jobs:
os: ubuntu-latest
TOX_EXTRA_COMMAND: "- isort --check-only --diff ."
TOXENV_SUFFIX: "-docs"
WITH_KROKI_CONTAINER: true
- python-version: "3.9"
os: ubuntu-latest
TOX_EXTRA_COMMAND: "- black --check --diff ./rdflib"
Expand All @@ -54,11 +52,9 @@ jobs:
os: ubuntu-latest
TOX_EXTRA_COMMAND: "flake8 --exit-zero rdflib"
TOXENV_SUFFIX: "-docs"
WITH_KROKI_CONTAINER: true
- python-version: "3.11"
os: ubuntu-latest
TOXENV_SUFFIX: "-docs"
WITH_KROKI_CONTAINER: true
steps:
- uses: actions/checkout@v3
- name: Cache XDG_CACHE_HOME
Expand Down Expand Up @@ -96,7 +92,6 @@ jobs:
EXTENSIVE=${{ matrix.extensive-tests || 'false' }} \
TOX_PYTHON_VERSION=${{ matrix.python-version }} \
TOXENV_SUFFIX=${{ matrix.TOXENV_SUFFIX }} \
WITH_KROKI_CONTAINER=${{ matrix.WITH_KROKI_CONTAINER || 'false' }} \
TOX_JUNIT_XML_PREFIX=${{ matrix.python-version }}-${{ matrix.os }}${{matrix.suffix}}- \
gha:validate
env:
Expand Down
8 changes: 0 additions & 8 deletions Taskfile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,6 @@ vars:
PIP_COMPILE: pip-compile
DOCKER: docker
OCI_REFERENCE: ghcr.io/rdflib/rdflib
WITH_KROKI_CONTAINER: "false"
KROKI_URL: "{{if (mustFromJson .WITH_KROKI_CONTAINER)}}http://localhost:15258/{{end}}"
tasks:
install:system-deps:
desc: Install system dependencies
Expand Down Expand Up @@ -98,7 +96,6 @@ tasks:
desc: Run tox
cmds:
- echo "TOXENV=${TOXENV}"
- echo "KROKI_URL=${KROKI_URL}"
- |
{{if .TOX_PYTEST_ARGS}}TOX_PYTEST_ARGS={{shellQuote .TOX_PYTEST_ARGS}}{{end}} \
{{if .TOX_EXTRA_COMMAND}}TOX_EXTRA_COMMAND={{shellQuote .TOX_EXTRA_COMMAND}}{{end}} \
Expand All @@ -109,7 +106,6 @@ tasks:
-m tox \
{{.CLI_ARGS}}
env:
KROKI_URL: '{{.KROKI_URL}}'
TOXENV: '{{if .TOX_PYTHON_VERSION}}py{{mustRegexReplaceAll "^([0-9]+)[.]([0-9]+).*" .TOX_PYTHON_VERSION "${1}${2}"}}{{if (mustFromJson .EXTENSIVE)}}-extensive{{end}}{{.TOXENV_SUFFIX | default ""}}{{end}}'
test:
desc: Run tests
Expand Down Expand Up @@ -179,8 +175,6 @@ tasks:

docs:
desc: Build documentation
env:
KROKI_URL: '{{.KROKI_URL}}'
cmds:
- echo "PYTHONPATH=${PYTHONPATH}"
- "{{.VENV_PYTHON}} -m sphinx.cmd.build -T -W -b html -d docs/_build/doctree docs docs/_build/html {{.CLI_ARGS}}"
Expand Down Expand Up @@ -257,8 +251,6 @@ tasks:
vars:
WITH_GITHUB_ACTIONS: true
- cmd: "{{.PYTHON}} -m pip install coveralls"
- cmd: "{{if (mustFromJson .WITH_KROKI_CONTAINER)}}docker compose up --detach kroki{{end}}"
- defer: "{{if (mustFromJson .WITH_KROKI_CONTAINER)}}docker compose down{{end}}"
- task: tox
vars:
COVERAGE_FILE: ".coverage"
Expand Down
4 changes: 0 additions & 4 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,5 @@ services:
volumes:
- .:/srv/workspace:z,cached
- workspace-var:/srv/workspace-var
kroki:
image: docker.io/yuzutech/kroki:latest
ports:
- 15258:8000
volumes:
workspace-var: {}
5 changes: 0 additions & 5 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,17 +44,12 @@
"sphinx.ext.ifconfig",
"sphinx.ext.viewcode",
"myst_parser",
"sphinxcontrib.kroki",
"sphinx.ext.autosectionlabel",
]

apidoc_module_dir = "../rdflib"
apidoc_output_dir = "apidocs"

if os.environ.get("KROKI_URL", ""):
kroki_url = os.environ["KROKI_URL"]
logging.warning(f"Using kroki_url={kroki_url}")

# https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html
autodoc_default_options = {"special-members": True}

Expand Down
Loading

0 comments on commit d3835e3

Please sign in to comment.