fix(deps): support semantic PII installation on Python 3.13 - #315
Conversation
Signed-off-by: Punleuk Oum <5661986+illeatmyhat@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe pull request adds Python 3.13 to the test matrix and requires ChangesPython compatibility
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The dependency and Python 3.13 matrix updates are consistent with the stated compatibility goal and are ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
The release workflow fails before tests on Python 3.13 because the semantic PII extra locks curated-tokenizers 0.0.9, which has no compatible wheel and crashes during Cython compilation.
Require curated-tokenizers >=0.0.10 and update only that package in the lockfile. The patch release supplies Python 3.13 wheels and satisfies the existing spacy-curated-transformers dependency constraints. Add Python 3.13 to the PR test matrix so the same frozen, all-extras installation is checked before release.
Validation:
Fixes the dependency-install failure in https://github.com/AgentToolkit/altk-evolve/actions/runs/34872870852. No retention behavior changes. This does not trigger publishing; a release can be retried after the fix is merged.
Summary by CodeRabbit
Tests
Chores