Skip to content

Install libvips in the pytest workflow to fix failing test job - #10

Merged
barrettMCW merged 2 commits into
mainfrom
copilot/fix-failing-github-actions-job-test
Oct 13, 2026
Merged

barrettMCW merged 2 commits into
mainfrom
copilot/fix-failing-github-actions-job-test

Conversation

Copilot AI commented Oct 13, 2026 •

Copy link
Copy Markdown

The test job failed because four tile-position tests in tests/test_shell.py load pyvips, which needs libvips.so.42. The ubuntu-latest runner doesn't ship that library, so the tests raised OSError: cannot load library 'libvips.so.42'.

  • .github/workflows/pytest.yml
    • Adds an "Install libvips" step before the Python dependency install.
    • Installs libvips42t64, the package name on Ubuntu 24.04, and falls back to libvips42 for older runner images.
- name: Install libvips
  run: |
    sudo apt-get update
    sudo apt-get install -y libvips42t64 || sudo apt-get install -y libvips42

The workflow hasn't been run with this change, so confirm the package name resolves on the current runner image.

Co-authored-by: barrettMCW <103449618+barrettMCW@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'test' Install libvips in the pytest workflow to fix failing test job Oct 13, 2026
Copilot AI requested a review from barrettMCW October 13, 2026 14:13
@barrettMCW
barrettMCW marked this pull request as ready for review October 13, 2026 14:21
Copilot AI balanced review requested due to automatic review settings October 13, 2026 14:21
@barrettMCW
barrettMCW merged commit b59fc7a into main Oct 13, 2026
4 of 7 checks passed
@barrettMCW
barrettMCW deleted the copilot/fix-failing-github-actions-job-test branch October 13, 2026 14:22
Copilot AI requested a review from barrettMCW October 13, 2026 14:23
Copilot stopped work on behalf of barrettMCW due to an error October 13, 2026 14:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The package name is valid for the current Ubuntu runner image, and the fallback safely supports older images.

0 open findings

What changed in this PR

Adds the native libvips dependency required by pyvips-based tests.

Changes:

  • Installs libvips42t64, with a fallback to libvips42.
  • Places installation before Python dependencies and pytest execution.
File Description
.github/​workflows/​pytest.yml Installs the required libvips runtime library.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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