Refactor: Remove Cython - Reduce tech debt

.github/workflows/docs.yml

@@ -1,72 +1,27 @@
+# This workflow is just to test that the docs build successfully.
 name: docs
-
+concurrency:
+ group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+ cancel-in-progress: true
 on:
- # Trigger manually
  workflow_dispatch:
-
- # Trigger on any push to the main
  push:
  branches:
  - main
- - development
-
- # Trigger on any push to a PR that targets main
  pull_request:
  branches:
  - main
- - development
-
 permissions:
  contents: write
-
-env:
- name: "ConfigSpace"
-
 jobs:
- build-and-deploy:
+ build:
  runs-on: ubuntu-latest
  steps:
- - name: Checkout
- uses: actions/checkout@v3
-
- - name: Setup Python
- uses: actions/setup-python@v3
- with:
- python-version: "3.8"
-
- - name: Install dependencies
- run: |
- pip install build
- pip install ".[docs]"
-
- - name: Make docs
- run: |
- make clean
- make docs
-
- - name: Pull latest gh-pages
- if: (contains(github.ref, 'develop') || contains(github.ref, 'main')) && github.event_name == 'push'
- run: |
- cd ..
- git clone https://github.com/${{ github.repository }}.git --branch gh-pages --single-branch gh-pages
-
- - name: Copy new docs into gh-pages
- if: (contains(github.ref, 'develop') || contains(github.ref, 'main')) && github.event_name == 'push'
- run: |
- branch_name=${GITHUB_REF##*/}
- cd ../gh-pages
- rm -rf $branch_name
- cp -r ../${{ env.name }}/docs/build/html $branch_name
-
- - name: Push to gh-pages
- if: (contains(github.ref, 'develop') || contains(github.ref, 'main')) && github.event_name == 'push'
- run: |
- last_commit=$(git log --pretty=format:"%an: %s")
- cd ../gh-pages
- branch_name=${GITHUB_REF##*/}
- git add $branch_name/
- git config --global user.name 'Github Actions'
- git config --global user.email 'not@mail.com'
- git remote set-url origin https://x-access-token:${{ secrets.GITHUB_TOKEN }}@github.com/${{ github.repository }}
- git commit -am "$last_commit"
- git push
+ - uses: actions/checkout@v4
+ - uses: actions/setup-python@v5
+ with:
+ python-version: "3.10"
+ - name: "Install dependancies"
+ run: python -m pip install -e ".[dev]"
+ - name: "Build Docs"
+ run: mkdocs build --clean --strict

.github/workflows/pytest.yml

@@ -32,11 +32,9 @@ env:
  --durations=20
  -v
 
-
 jobs:
+ test:
 
- # General unit tests
- source-test:
  name: ${{ matrix.python-version }}-${{ matrix.os }}
 
  runs-on: ${{ matrix.os }}
@@ -49,12 +47,6 @@ jobs:
  matrix:
  python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
  os: ["ubuntu-latest", "macos-latest", "windows-latest"]
- exclude:
- # These don't seem to be available anymore with setup-python?
- - python-version: "3.8"
- os: "macos-latest"
- - python-version: "3.9"
- os: "macos-latest"
 
  steps:
  - name: Checkout
@@ -64,13 +56,13 @@ jobs:
  uses: actions/setup-python@v5
  with:
  python-version: ${{ matrix.python-version }}
+ cache: pip
+ cache-dependency-path: pyproject.toml
 
  - name: Install ${{ env.package-name }}
  run: |
- python -m pip install --upgrade pip
  python -m pip install -e ".${{ env.extra-requires }}"
 
  - name: Tests
- timeout-minutes: 45
  run: |
- pytest ${{ env.pytest-args }} ${{ env.test-dir }}
+ pytest ${{ env.test-dir }}

Makefile

@@ -6,12 +6,13 @@
 
 .PHONY: help install-dev install-test install-docs pre-commit clean clean-doc clean-build build docs links publish test clean-test
 
+A_DIFFERENT_CHANGE
+
 help:
  @echo "Makefile ConfigSpace"
  @echo "* install-dev to install all dev requirements and install pre-commit"
- @echo "* pre-commit to run the pre-commit check"
+ @echo "* check  to run the pre-commit check"
  @echo "* docs to generate and view the html files"
- @echo "* linkcheck to check the documentation links"
  @echo "* publish to help publish the current branch to pypi"
  @echo "* test to run the tests"
 
@@ -39,48 +40,20 @@ install-dev:
  $(PIP) install -e ".[dev]"
  pre-commit install
 
-install-test:
- $(PIP) install -e ".[test]"
-
-install-docs:
- $(PIP) install -e ".[docs]"
-
 check:
  $(PRECOMMIT) run --all-files
 
-check-types:
- mypy ConfigSpace
-
 fix:
  black --quiet ConfigSpace test
  ruff --silent --exit-zero --no-cache --fix ConfigSpace test
 
-build:
- python -m build
-
 test:
  $(PYTEST) test
 
-clean-build:
- rm -rf ${BUILD}
-
-clean-docs:
- $(MAKE) -C ${DOCDIR} clean
-
-clean: clean-build clean-docs
-
-clean-test: clean-build build test
-
 # Running build before making docs is needed all be it very slow.
 # Without doing a full build, the doctests seem to use docstrings from the last compiled build
 docs: clean build
- $(MAKE) -C ${DOCDIR} html
- @echo
- @echo "View docs at:"
- @echo ${INDEX_HTML}
-
-links:
- $(MAKE) -C ${DOCDIR} linkcheck
+ mkdocs serve
 
 # Publish to testpypi
 # Will echo the commands to actually publish to be run to publish to actual PyPi