Skip to content

Deterministic ci - #311

Merged
seddonym merged 3 commits into
mainfrom
deterministic-ci
Aug 7, 2026
Merged

Deterministic ci#311
seddonym merged 3 commits into
mainfrom
deterministic-ci

Conversation

@seddonym

@seddonym seddonym commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Prior to this, the CI runs weren't deterministic:

  • The Python libraries weren't guaranteed to be the same (they would update automatically) due to no uv.lock file.
  • The Ruff toolchain wasn't guaranteed to be the same.

This adds a uv.lock file which was added to .gitignore in error, and a rust-toolchain.toml to pin the rust version. It also fixes some minor issues picked by by clippy under the latest toolchain.

We should still commit uv.lock, otherwise CI is not reproducible.
Prior to this, these lines caused errors when linting rust.
This makes CI more deterministic.
@codspeed-hq

codspeed-hq Bot commented Aug 7, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 26 untouched benchmarks
⏩ 23 skipped benchmarks1


Comparing deterministic-ci (6a45cdd) with main (00bd8b5)

Open in CodSpeed

Footnotes

  1. 23 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@seddonym
seddonym merged commit faab22b into main Aug 7, 2026
18 checks passed
@seddonym
seddonym deleted the deterministic-ci branch August 7, 2026 10:56
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.

1 participant