Build in debug mode for PRs #1375
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ding the wheel artifacts from build stage
…pushes to main and release or candidate tags.
…s erasing the other wheel
martin-g
reviewed
Feb 9, 2026
Co-authored-by: Martin Grigorov <martin-g@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors CI to use reusable build/test workflows and to run PR builds in debug mode for faster feedback while keeping release-mode builds for main and tags.
Changes:
- Split CI into reusable workflows (
build.yml,test.yml) and added dedicated entry workflows for PRs (ci.yml) and releases (release.yml). - Updated test workflow to install and test against the pre-built Linux wheel artifact instead of rebuilding during tests.
- CI/config formatting cleanups across TOML files and adjusted license generation to rely on
cargo-license.
Reviewed changes
Copilot reviewed 9 out of 11 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| pyproject.toml | Formatting cleanup and a dependency list syntax fix (trailing comma). |
| examples/datafusion-ffi-example/Cargo.toml | TOML formatting cleanup for pyo3 features list. |
| examples/datafusion-ffi-example/.cargo/config.toml | TOML formatting cleanup for rustflags. |
| dev/create_license.py | Switches license generation to call cargo-license directly. |
| .github/workflows/test.yml | Converts to reusable workflow and installs pre-built wheel artifacts for tests. |
| .github/workflows/release.yml | Adds release entry workflow calling reusable build/test in release mode. |
| .github/workflows/ci.yml | Adds PR entry workflow calling reusable build/test in debug mode. |
| .github/workflows/build.yml | Converts to reusable workflow; adds linting jobs and reworks wheel build/artifact behavior. |
| .cargo/config.toml | TOML formatting cleanup for rustflags. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is to improve our CI performance in a couple of ways