From ea8c349f6dce955696850198b8544d0203b467fb Mon Sep 17 00:00:00 2001
From: Roberto Bertó <463349+robertoberto@users.noreply.github.com>
Date: Tue, 19 May 2026 02:35:08 +0000
Subject: chore: clean packaging and development tooling
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This commit modernizes the project's tooling and packaging without
changing the runtime code. It addresses dead workflows, obsolete
helper scripts, duplicated configuration, and stale developer docs.

Removed:
- .github/workflows/python-app.yml: targeted Python 3.12, referenced a
  non-existent requirements.txt, ran only flake8 with pytest commented out
- Makefile: hard-coded env/bin/python paths that do not work with uv
- run_tests.sh and run_tests.py: duplicated each other and referenced
  removed modules (test_exceptions, test_quick)
- sphinx/ and .readthedocs.yaml: the RTD config pointed to
  docs/source/conf.py while the sphinx tree lived under sphinx/source,
  so the build never worked and no docs were ever published
- docs/development/architecture.md, refactor-roadmap.md, and
  quality-and-utils.md: described the pre-cleanup proposal that
  included specs/exceptions/request_id, now contradicted by the code

Edited:
- pyproject.toml:
  - dropped the validation extra (Pydantic) — specs/ is gone
  - dropped the duplicated [tool.hatch.metadata].dependencies block
  - dropped the duplicated [dependency-groups].dev block
  - declared the wheel package explicitly via
    [tool.hatch.build.targets.wheel].packages
  - bumped pytest floor to >=8.0 (Python 3.13 compatible)
  - added richer classifiers (Development Status, audience, topic,
    Typing :: Typed), keywords, license file pointer, and a Changelog URL
- .github/workflows/python-pr-validation.yml: upgraded to
  actions/checkout@v4 and setup-python@v5, switched to astral-sh/setup-uv,
  removed obsolete architecture argument
- .github/dependabot.yml: added the github-actions ecosystem so
  workflow versions stay current

Added:
- pyvyos/py.typed: PEP 561 marker advertising the package as typed
- .pre-commit-config.yaml: neutral hooks only (whitespace, EOF, YAML/TOML
  syntax, large-file guard); no formatters or linters yet

Kept:
- docs/development/vyos_api/: JSON reference for the VyOS HTTPS API,
  useful for future contract tests

Tests still pass: 57/57.
---
 .github/workflows/python-app.yml | 39 ---------------------------------------
 1 file changed, 39 deletions(-)
 delete mode 100644 .github/workflows/python-app.yml

(limited to '.github/workflows/python-app.yml')

diff --git a/.github/workflows/python-app.yml b/.github/workflows/python-app.yml
deleted file mode 100644
index 96d39fa..0000000
--- a/.github/workflows/python-app.yml
+++ /dev/null
@@ -1,39 +0,0 @@
-# This workflow will install Python dependencies, run tests and lint with a single version of Python
-# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python
-
-name: Python application
-
-on:
-  push:
-    branches: [ "main" ]
-  pull_request:
-    branches: [ "main" ]
-
-permissions:
-  contents: read
-
-jobs:
-  build:
-
-    runs-on: ubuntu-latest
-
-    steps:
-    - uses: actions/checkout@v4.1.1
-    - name: Set up Python 3.12.2
-      uses: actions/setup-python@v5.0.0
-      with:
-        python-version: "3.12.2"
-    - name: Install dependencies
-      run: |
-        python -m pip install --upgrade pip
-        pip install flake8 pytest
-        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
-    - name: Lint with flake8
-      run: |
-        # stop the build if there are Python syntax errors or undefined names
-        flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
-        # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
-        flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
-    #- name: Test with pytest
-    #  run: |
-    #    pytest
-- 
cgit v1.2.3