Skip to content

ci: llama.cpp - simplify test setup#3113

Merged
anakin87 merged 9 commits intomainfrom
llama-cpp-fix-test
Apr 7, 2026
Merged

ci: llama.cpp - simplify test setup#3113
anakin87 merged 9 commits intomainfrom
llama-cpp-fix-test

Conversation

@anakin87
Copy link
Copy Markdown
Member

@anakin87 anakin87 commented Apr 7, 2026

Related Issues

The problem is we install macOS packages from an additional index (not PyPI) to prevent building errors, but the latest version is broken: ZIP file contains trailing contents after the end-of-central-directory

Proposed Changes:

  • simplify test/CI setup
    • do not use additional indexes: the build errors have been fixed
    • test on macOS with Python 3.10 and 3.14 (like other operating systems)

How did you test it?

CI

Checklist

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 7, 2026

Coverage report (llama_cpp)

This PR does not seem to contain any modification to coverable code.

@anakin87 anakin87 changed the title test: fix llama.cpp test: llama.cpp - do not install broken 0.3.20 on macOs Apr 7, 2026
@anakin87 anakin87 changed the title test: llama.cpp - do not install broken 0.3.20 on macOs test: llama.cpp - do not install broken 0.3.20 on macOS Apr 7, 2026
@anakin87 anakin87 changed the title test: llama.cpp - do not install broken 0.3.20 on macOS ci: llama.cpp - simplify test setup Apr 7, 2026
@anakin87 anakin87 marked this pull request as ready for review April 7, 2026 14:21
@anakin87 anakin87 requested a review from a team as a code owner April 7, 2026 14:21
@anakin87 anakin87 requested review from davidsbatista and removed request for a team April 7, 2026 14:21
@anakin87 anakin87 self-assigned this Apr 7, 2026
@anakin87 anakin87 merged commit 1948dd4 into main Apr 7, 2026
13 checks passed
@anakin87 anakin87 deleted the llama-cpp-fix-test branch April 7, 2026 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants