Skip to content

ci: release build_tests install the test extra - #188

Merged
JarbasAl merged 1 commit into
devfrom
ci/release-build-tests-use-test-extra
Sep 16, 2026
Merged

JarbasAl merged 1 commit into
devfrom
ci/release-build-tests-use-test-extra

Conversation

@JarbasAl

@JarbasAl JarbasAl commented Sep 7, 2026

Copy link
Copy Markdown
Member

🤖 Auto-generated by Claude Fable 5.1 (claude-fable-5-1) via Claude Code — NOT human-reviewed. Verify before acting.

The release-workflow.yml build_tests job was configured to install only the "audio,pydantic" extras, which is narrower than what the test suite requires. Two test modules import optional dependencies not included in that set: test_golden.py needs the engines extra (nebulento, ovos-m2v-pipeline), and test_listener_stream.py needs the listener extra (ovos-dinkum-listener). This configuration gap was not discovered until the first manual workflow_dispatch run on 2026-09-07 21:29 UTC, which failed 8 tests across all four Python version legs (3.10, 3.11, 3.12, 3.13).

This change aligns the release workflow's build_tests job with the PR-check workflow's build-tests.yml, both now installing the full test extra. The test extra is the superset that covers all test dependencies (audio, pydantic, listener, engines, and the TTS/media/bench utilities), verified in pyproject.toml. Installing test instead of the narrower audio,pydantic set resolves the missing optional dependencies without introducing any new packages beyond what the test suite already expects.

The release-workflow.yml build_tests job was using install_extras: "audio,pydantic",
which is narrower than the extras needed by the test suite. test_golden.py requires
the engines extra (nebulento, ovos-m2v-pipeline), and test_listener_stream.py
requires the listener extra (ovos-dinkum-listener). Align with build-tests.yml by
installing the full test extra, which is the superset that covers all test dependencies.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Elementary, my dear contributor! The checks are finished. 🕵️‍♂️

I've aggregated the results of the automated checks for this PR below.

📋 Repo Health

Ensuring the repository stays strong and healthy. 💪

✅ All required files present.

Latest Version: 1.8.1a1

ovoscope/version.py — Version file
README.md — README
LICENSE — License file
pyproject.toml — pyproject.toml
⚠️ setup.py — setup.py
CHANGELOG.md — Changelog
ovoscope/version.py has valid version block markers

🏷️ Release Preview

I've checked the 'New Features' highlight reel. 📽️

Current: 1.8.1a1Next: 1.8.1a2

Signal Value
Label (none)
PR title ci: release build_tests install the test extra
Bump alpha

✅ PR title follows conventional commit format.


🚀 Release Channel Compatibility

Predicted next version: 1.8.1a2

Channel Status Note Current Constraint
Stable Not in channel -
Testing Too new (must be <1.0.0) ovoscope>=0.7.2,<1.0.0
Alpha Compatible ovoscope>=1.6.15a1

🔍 Lint

Checking if we've met all our check criteria. ✅

ruff: issues found — see job log

🔒 Security (pip-audit)

Ensuring our cookies are secure and fresh. 🍪

✅ No known vulnerabilities found (129 packages scanned).

⚖️ License Check

Scanning for any hidden proprietary blobs. 🌑

✅ No license violations found.

Policy: Apache 2.0 (universal donor). StrongCopyleft / NetworkCopyleft / WeakCopyleft / Other / Error categories fail. MPL allowed.

📊 Coverage

A deep dive into the sea of test results. 🌊

⚠️ 62.8% total coverage

Files below 80% coverage (15 files)
File Coverage Missing lines
ovoscope/simple_listener.py 0.0% 63
ovoscope/version.py 0.0% 5
ovoscope/classic_listener.py 18.2% 117
ovoscope/intent_cases.py 22.3% 153
ovoscope/pytest_plugin.py 47.0% 210
ovoscope/cli.py 47.8% 132
ovoscope/ocp.py 47.9% 61
ovoscope/tts_intelligibility.py 49.0% 102
ovoscope/e2e.py 56.2% 67
ovoscope/media.py 56.7% 101
ovoscope/listener.py 57.1% 127
ovoscope/voice_loop.py 59.0% 119
ovoscope/__init__.py 63.2% 423
ovoscope/audio.py 64.4% 126
ovoscope/media_provider.py 67.6% 23

Full report: download the coverage-report artifact.

🔨 Build Tests

Testing the assembly line for your latest changes. 🏭

✅ All versions pass

Python Build Install Tests
3.10
3.11
3.12
3.13
3.14

Your 24/7 automated code reviewer 🌙

@JarbasAl
JarbasAl marked this pull request as ready for review September 16, 2026 19:10
@JarbasAl
JarbasAl merged commit 57804ec into dev Sep 16, 2026
14 checks passed
@JarbasAl
JarbasAl deleted the ci/release-build-tests-use-test-extra branch September 16, 2026 19:10
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.

1 participant