Conversation
| docker stop llmster || true | ||
| docker rm llmster || true | ||
|
|
||
| # Coverage check for Docker tests (higher target since they include LM Studio tests) |
There was a problem hiding this comment.
Only one Python version per-platform actually collects coverage data, so if we're not combining across platforms, there's no need for the coverage analysis to be a separate job.
I'm not sure of the rationale for skipping the combined coverage report, though. Checking the per-platform coverage in addition to the combined coverage does seem reasonable.
|
@rugvedS07 One of your remaining failures is an existing test ordering issue: the intended usage of
Probably the simplest interim fix is to pre-download (but not load) The other failure I saw is due to one of the plugin test cases not seeing the expected graceful shutdown message. It's plausible that one is just outright flaky and/or distro sensitive (it's previously only been tested by me locally in Fedora WSL and Windows Powershell) |
|
I just added Python 3.14 to the CI test matrix in main, so that's the origin of the reported merge conflict. |
Overview
Adds the ability to run full tests in CI for linux using the docker container.
Fixes #140 for linux tests