Skip to content

ci: fix build and modernize CI/CD workflows #117

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JarbasAl merged 4 commits into from
Apr 25, 2026
Merged

ci: fix build and modernize CI/CD workflows #117

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
146a9b6
ci: add pyproject.toml and fix build to unblock CI
EggmanBot Apr 24, 2026
bc0c6a4
chore: migrate to pyproject.toml, drop setup.py and requirements.txt
EggmanBot Apr 24, 2026
085efe9
fix: correct pyproject.toml flat-layout package config to fix CI build
EggmanBot Apr 25, 2026
ab1acc1
fix: resolve remaining CI failures — missing deps and stale test imports
EggmanBot Apr 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions MANIFEST.in
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
include requirements.txt
recursive-include dialog *
recursive-include vocab *
recursive-include locale *
Expand Down
3 changes: 3 additions & 0 deletions pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
[build-system]
requires = ["setuptools>=42", "wheel"]
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Apr 25, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🌐 Web query:

What is the minimum setuptools version that supports recursive ** glob patterns in package-data / package_data?

💡 Result:

The minimum setuptools version that supports recursive glob patterns (using ) in package_data / package_data is 62.3.0.

Citations:

🏁 Script executed:

cat pyproject.toml

Repository: OpenVoiceOS/ovos-skill-volume

Length of output: 1387

Bump setuptools minimum to >=62.3.0 — recursive ** globs in package-data require this version.

The package-data entries (locale/**/*, ui/**/*, vocab/**/*, dialog/**/*, regex/**/*, skill/**/*) use recursive glob patterns that were added to setuptools in version 62.3.0. With setuptools>=42, older build environments that resolve setuptools <62.3.0 will silently ignore these patterns, causing locale, ui, vocab, dialog, regex, and skill assets to be missing from both sdist and wheel — re-introducing the exact FileNotFoundError packaging issues this PR aims to fix.

Additionally, ovos-workshop>=8.0.0,<9.0.0 appears in both dependencies (line 20) and the test extra (line 28), which is redundant since it is already available at runtime.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@pyproject.toml` at line 2, Update pyproject.toml to require
setuptools>=62.3.0 instead of >=42 so the recursive glob patterns in
package-data (locale/**, ui/**, vocab/**, dialog/**, regex/**, skill/**) are
honored during sdist/wheel builds; also remove the duplicate
ovos-workshop>=8.0.0,<9.0.0 entry from the "test" extra (keep it in the main
dependencies) so it’s not redundantly listed. Use the existing "requires" key to
bump the setuptools version and edit the extras[test] block to delete the
ovos-workshop line.

build-backend = "setuptools.build_meta"
2 changes: 2 additions & 0 deletions version.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,5 @@
VERSION_BUILD = 21
VERSION_ALPHA = 2
# END_VERSION_BLOCK

__version__ = f"{VERSION_MAJOR}.{VERSION_MINOR}.{VERSION_BUILD}" + (f"a{VERSION_ALPHA}" if VERSION_ALPHA else "")
Loading