Skip to content

Fix CI issues (flaky Python tests & missing rustfmt)#186

Merged
simu merged 2 commits intomainfrom
fix/ci
Oct 1, 2025
Merged

Fix CI issues (flaky Python tests & missing rustfmt)#186
simu merged 2 commits intomainfrom
fix/ci

Conversation

@simu
Copy link
Copy Markdown
Member

@simu simu commented Sep 25, 2025

Follow-up for #183 to actually use updated Rust toolchain for Python tests on x86_64. Additionally also install missing rustfmt component for the "lints" GitHub workflow.

Checklist

  • The PR has a meaningful title. The title will be used to auto generate the changelog
  • PR contains a single logical change (to build a better changelog).
  • Update tests.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency, internal
    as they show up in the changelog
  • Link this PR to related PRs or issues.

And make rust version selection more verbose so we can debug potential
future issues easier.
@simu simu requested a review from a team September 25, 2025 15:02
@simu simu added the ignore label Sep 25, 2025
@simu simu changed the title Fix/ci Fix CI issues (flaky Python tests & missing rustfmt) Sep 25, 2025
@github-actions
Copy link
Copy Markdown

Benchmark for f25fae4

Click to view benchmark
Test Base PR %
Reclass::inventory() multi-threaded 1687.1±49.05µs 1727.3±165.27µs +2.38%
Reclass::inventory() single-threaded 3.6±0.02ms 3.5±0.05ms -2.78%

@simu simu merged commit ba9532d into main Oct 1, 2025
49 of 56 checks passed
@simu simu deleted the fix/ci branch October 1, 2025 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants