Skip to content

Modernize CI: GitHub Actions, codecov, fix deprecated assertEquals#44

Merged
mahmoud merged 2 commits into
masterfrom
modernize-ci
Jun 29, 2026
Merged

Modernize CI: GitHub Actions, codecov, fix deprecated assertEquals#44
mahmoud merged 2 commits into
masterfrom
modernize-ci

Conversation

@mahmoud

@mahmoud mahmoud commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Changes

  • GitHub Actions workflow (.github/workflows/ci.yml)

    • actions/setup-python@v5 with built-in cache: pip (no manual actions/cache)
    • codecov/codecov-action@v5 with CODECOV_TOKEN
    • Matrix: Python 3.9–3.14 + PyPy 3.10
    • paths-ignore for docs/markdown
    • fail_ci_if_error: true
  • codecov.yml — project threshold 0.5%, patch target 100%

  • Test fixassertEqualsassertEqual (8 call sites in tests/template_loaders.py), removed in Python 3.12+

Replaces the legacy .travis.yml (Python 2.6/2.7/3.3/3.4).

mahmoud added 2 commits June 28, 2026 21:09
- Add GitHub Actions workflow (.github/workflows/ci.yml)
  - actions/setup-python@v5 with built-in cache: pip
  - codecov/codecov-action@v5 with CODECOV_TOKEN
  - Matrix: Python 3.9-3.14 + PyPy 3.10
  - paths-ignore for docs/markdown
- Add codecov.yml with project/patch coverage thresholds
- Fix assertEquals -> assertEqual in tests (removed in Python 3.12+)
setup-python's cache: pip requires requirements.txt or pyproject.toml
by default. This project uses bare setup.py, so point it explicitly.
@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@mahmoud mahmoud merged commit 50fee8d into master Jun 29, 2026
14 of 28 checks passed
@mahmoud mahmoud deleted the modernize-ci branch June 29, 2026 04:22
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