Skip to content

Faster build#27

Draft
apaetz wants to merge 22 commits into
mainfrom
apaetz/faster-build
Draft

Faster build#27
apaetz wants to merge 22 commits into
mainfrom
apaetz/faster-build

Conversation

@apaetz

@apaetz apaetz commented Feb 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

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 aims to speed up builds in the Rust workspace (and CI) by adjusting test/build configuration for the PyO3-based Python binding crates and by tuning CPU-specific compilation flags.

Changes:

  • Disable the lib test harness for the binar and paulimer Python binding crates via test = false.
  • Update GitHub Actions workflow steps (formatting, cargo test --all-targets, and Python bindings build) and set RUSTFLAGS for bindings builds.
  • Change workspace-wide Cargo rustflags to -C target-cpu=native, and add arch-aware RUSTFLAGS selection in the ADO pipeline for wheel builds.

Reviewed changes

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

Show a summary per file
File Description
paulimer/bindings/python/Cargo.toml Disables the lib test harness for the paulimer Python binding crate.
binar/bindings/python/Cargo.toml Disables the lib test harness for the binar Python binding crate.
.github/workflows/build.yaml Reorders CI steps, updates cargo test flags, and consolidates Python bindings builds with explicit RUSTFLAGS.
.cargo/config.toml Switches global workspace rustflags to target-cpu=native.
.ado/stages/build.yaml Adds architecture-based RUSTFLAGS selection for wheel builds (x86_64 vs ARM64).

Comment thread .github/workflows/build.yaml Outdated
Comment thread .github/workflows/build.yaml Outdated
Comment thread .cargo/config.toml
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