Skip to content

ci: run the unit tests when data or config change - #706

Draft
openvoiceos-bot wants to merge 1 commit into
devfrom
ci/unit-tests-data-paths
Draft

openvoiceos-bot wants to merge 1 commit into
devfrom
ci/unit-tests-data-paths

Conversation

@openvoiceos-bot

Copy link
Copy Markdown
Contributor

🤖 Auto-generated by Claude Opus 5.5 (claude-opus-5-5) via Claude Code — NOT human-reviewed. Verify before acting.

The unit test workflow runs when data/** or config/** changes, on push and on pull request.

The unit tests read data/, and the scripts that generate data/ read config/. The paths: filter did not list either directory, so a pull request that changes only data started no unit test run. ovos-localize#705, which regenerates data/locale_rules.json, shows this: it has no unit test check.

Item Value
File .github/workflows/unit_tests.yml
Change 'data/**' and 'config/**' added to both paths: lists
Tests that read data/ test_intents_corpus.py, test_check_path_lang.py, test_publish_hf_dataset.py, test_locale_rules.py

🤖 Generated with Claude Code

The unit tests read data/, and the scripts that generate data/ read
config/, so a change in either directory can break the tests. The
paths filter did not list them, and a pull request that changed only
data started no unit test run.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 9, 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
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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 Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

I've completed my sweep! Here's the situation. 🧹

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

🏷️ Release Preview

The draft for the big day is ready for review. 📝

Current: 0.1.0a27 → Next: 0.1.0a28

Signal Value
Label (none)
PR title ci: run the unit tests when data or config change
Bump alpha

✅ PR title follows conventional commit format.


🚀 Release Channel Compatibility

Predicted next version: 0.1.0a28

Channel Status Note Current Constraint
Stable ⚪ Not in channel -
Testing ⚪ Not in channel -
Alpha ⚪ Not in channel -

🔍 Lint

Checking if there's anything else we need to do. 📋

❌ ruff: issues found in 83 file(s) — see job log
✅ actionlint: 12 workflow file(s), no finding

🔨 Build Tests

Ensuring the code is correctly packaged and ready. 📦

Python Build Install Tests pytest
3.10 ✅ ✅ ⚠️ 1 failed, 580 passed in 5.86s
3.11 ✅ ✅ ⚠️ 1 failed, 580 passed, 2 warnings in 8.91s
3.12 ✅ ✅ ⚠️ 1 failed, 580 passed, 2 warnings in 9.35s
3.13 ✅ ✅ ⚠️ 1 failed, 580 passed, 1 warning in 6.19s
3.14 ✅ ✅ ⚠️ 1 failed, 580 passed, 1 warning in 10.49s

❌ 3.10: Install OK, tests failed
❌ 3.11: Install OK, tests failed
❌ 3.12: Install OK, tests failed
❌ 3.13: Install OK, tests failed
❌ 3.14: Install OK, tests failed
Check job logs for details.


Crafting a better voice assistant, one commit at a time 🎙️

This branch has not been deployed

No deployments
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.

2 participants