Skip to content

Conversation

@jdesboeufs
Copy link
Member

This PR modernizes the addok-sqlite-store repository to align with current Python packaging standards and development practices.

Changes

Packaging

  • Migrate to pyproject.toml: Replace legacy setup.py with modern declarative configuration using setuptools ≥65.0
  • Update Python support: Drop Python 3.4-3.6, add support for Python 3.9-3.14
  • Remove MANIFEST.in: No longer needed with modern pyproject.toml configuration
  • Add development dependencies: pytest ~8.3, pytest-cov ~6.1, build ~1.2, twine 6.1.0

CI/CD

  • Add GitHub Actions workflow: Automated testing on push and pull requests
  • Python matrix testing: Test against Python 3.9 and 3.14
  • Redis service integration: Proper Redis setup for integration tests
  • Coverage reporting: pytest-cov integration for test coverage

Development

  • Add Makefile: Standardized commands for development workflow
    • make develop: Install package in development mode
    • make test: Run test suite
    • make testcoverage: Run tests with coverage report
    • make dist: Build distribution packages
    • make upload: Upload to PyPI
    • make clean: Clean build artifacts

Documentation

  • Improve README: Add features section, installation instructions, and consistent formatting
  • Update LICENSE: Change copyright holder to "République Française"

Housekeeping

  • Update .gitignore: Add dist/ and venv/ entries

Compatibility

This update maintains backward compatibility for users of the package. The plugin continues to work the same way, with the same configuration options and entry points.

- Migrate from setup.py to pyproject.toml with modern setuptools
- Update Python support to 3.9-3.14
- Add GitHub Actions CI workflow with Redis service and Python matrix testing
- Add Makefile with development and distribution commands
- Update LICENSE copyright to République Française
- Improve README with features section and consistent formatting
- Remove obsolete MANIFEST.in and setup.py
- Update .gitignore for dist/ and venv/
@jdesboeufs jdesboeufs merged commit 5ff5538 into main Nov 1, 2025
2 checks passed
@jdesboeufs jdesboeufs deleted the modernize branch November 1, 2025 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants