Skip to content

Update CI actions#184

Merged
jbarry-bdai merged 8 commits intorai-opensource:masterfrom
petercorke:fix/python-eol
Apr 30, 2026
Merged

Update CI actions#184
jbarry-bdai merged 8 commits intorai-opensource:masterfrom
petercorke:fix/python-eol

Conversation

@petercorke
Copy link
Copy Markdown
Collaborator

@petercorke petercorke commented Apr 30, 2026

  • update Python used for sphinx build, coverage, testing
  • update version of actions used.

CI: update workflow actions and fix test environment

  • actions/checkout@v2 → @v4 in all three workflows — v2 is deprecated and has known security and compatibility issues
  • MPLBACKEND: TkAgg → Agg in the test job — TkAgg requires a display server and is unreliable on headless runners; Agg is the correct non-interactive backend for CI
  • Python 3.8 → 3.12 in the codecov job and publish.yml — Python 3.8 reached end-of-life in October 2024
  • To make this work, I had to do another round of changes. The Agg backend (for headless graphics) was hanging in the plt.pause() so added logic to skip certain unit tests in the CI environment.

Remaining known issues (tracked separately):

  • codecov/codecov-action@v3 should be updated to @v4
  • ad-m/github-push-action@master in sphinx.yml should be pinned to a specific commit SHA rather than a branch to prevent supply-chain risk
  • ubuntu-22.04 is hardcoded; consider ubuntu-latest
  • The windows-latest exclusions for Python 3.11 and 3.12 lack a comment explaining the reason

- update Python used for sphinx build, coverage, testing
- update version of actions used.
- skips tests that run forever, interaction of plt.pause() with Agg backend.
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@jbarry-bdai jbarry-bdai merged commit 5a52e75 into rai-opensource:master Apr 30, 2026
13 checks passed
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.

3 participants