Skip to content

Add Python CI, update API docs, and fix Rust append behavior#81

Merged
mathleur merged 4 commits intofeature/rustfrom
feat/improvements
Mar 24, 2026
Merged

Add Python CI, update API docs, and fix Rust append behavior#81
mathleur merged 4 commits intofeature/rustfrom
feat/improvements

Conversation

@mathleur
Copy link
Copy Markdown
Member

Description

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates CI and documentation while addressing a Rust Qube::append edge case when appending into an empty receiver.

Changes:

  • Add an empty-receiver fast-path to Qube::append and a regression test covering append-to-empty behavior.
  • Expand Rust and Python API documentation to include additional manipulation methods (drop, squeeze, etc.).
  • Update GitHub Actions CI to run Python binding tests inside an isolated uv-managed virtual environment.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
qubed/src/merge.rs Adds an is_empty() fast-path for append by copying other into self and clearing other.
qubed/tests/test_union.rs Adds a test for appending into an empty Qube; removes commented-out code.
docs/src/rust/qubed.md Documents additional Rust API methods and adds examples for drop/squeeze.
docs/src/python/py_qubed.md Adds Python docs for compress, drop, squeeze, and query methods.
.github/workflows/ci.yml Runs Python tests in a dedicated uv venv rather than the global runner environment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread qubed/src/merge.rs
Comment thread qubed/tests/test_union.rs Outdated
Comment thread docs/src/rust/qubed.md
Comment thread docs/src/python/py_qubed.md
mathleur and others added 3 commits March 20, 2026 14:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mathleur mathleur merged commit 3306335 into feature/rust Mar 24, 2026
1 check failed
@mathleur mathleur deleted the feat/improvements branch March 24, 2026 08:13
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