Skip to content

chore(deps): bump rand_core to 0.10.0-rc-5#108

Merged
fjarri merged 2 commits intoentropyxyz:masterfrom
baloo:baloo/push-zlqkpkvxqksw
Jan 20, 2026
Merged

chore(deps): bump rand_core to 0.10.0-rc-5#108
fjarri merged 2 commits intoentropyxyz:masterfrom
baloo:baloo/push-zlqkpkvxqksw

Conversation

@baloo
Copy link
Copy Markdown
Contributor

@baloo baloo commented Jan 19, 2026

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.93%. Comparing base (41b60a9) to head (0101887).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   97.93%   97.93%   -0.01%     
==========================================
  Files          14       14              
  Lines        1893     1891       -2     
==========================================
- Hits         1854     1852       -2     
  Misses         39       39              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tarcieri
Copy link
Copy Markdown
Contributor

Note: I'll have a crypto-bigint update out here shortly

@baloo baloo force-pushed the baloo/push-zlqkpkvxqksw branch from d7ce29a to 047afeb Compare January 19, 2026 21:07
@baloo
Copy link
Copy Markdown
Contributor Author

baloo commented Jan 19, 2026

I don't know if we need a bump of crypto-bigint. I think this is enough, at least it's enough for RSA to bump to rand_core 0.10.0-rc-4

@tarcieri
Copy link
Copy Markdown
Contributor

tarcieri commented Jan 19, 2026

I'm maybe a half hour away.

I think it's good if the versions are in lockstep. There hopefully aren't any breaking changes but it'd be good to know for sure.

@tarcieri
Copy link
Copy Markdown
Contributor

tarcieri commented Jan 19, 2026

It's out: https://crates.io/crates/crypto-bigint/0.7.0-rc.19

Hopefully doesn't break anything but it'd be good to know and get included along with bumping everything else.

I can try it out on your branch if you want.

(I can look in a bit, seems it broke something in elliptic-curve)

@tarcieri
Copy link
Copy Markdown
Contributor

Ugh, ran into some breakages in crypto-bigint and I'll be cutting another release

@baloo
Copy link
Copy Markdown
Contributor Author

baloo commented Jan 19, 2026

There is some amount of breakage in crypto-bigint I have to spend some time on it.

@baloo
Copy link
Copy Markdown
Contributor Author

baloo commented Jan 19, 2026

Ugh, ran into some breakages in crypto-bigint and I'll be cutting another release

ha.

@tarcieri
Copy link
Copy Markdown
Contributor

crypto-bigint v0.7.0-rc.20 is out

@baloo
Copy link
Copy Markdown
Contributor Author

baloo commented Jan 20, 2026

For some reason it would not find the AsRef<[Limb]> implementation anymore (or it had multiple alternative AsRef), I've switched to using as_limbs instead.

@tarcieri
Copy link
Copy Markdown
Contributor

@baloo this release added an overlapping impl: AsRef<UintRef> (and AsMut<UintRef>) so inference no longer works like it did before, unfortunately.

Maybe we should remove the AsRef<[Limb]> impl and people can either use .as_limbs() or .as_ref().as_limbs().

@baloo baloo force-pushed the baloo/push-zlqkpkvxqksw branch 2 times, most recently from 68aec46 to 266af98 Compare January 20, 2026 15:23
@baloo baloo changed the title chore(deps): bump rand_core to 0.10.0-rc-4 chore(deps): bump rand_core to 0.10.0-rc-5 Jan 20, 2026
@baloo baloo marked this pull request as ready for review January 20, 2026 15:23
@baloo baloo force-pushed the baloo/push-zlqkpkvxqksw branch from 266af98 to c9428e3 Compare January 20, 2026 15:31
@baloo baloo force-pushed the baloo/push-zlqkpkvxqksw branch from c9428e3 to bf0cecb Compare January 20, 2026 17:35
Comment thread Cargo.toml Outdated
Comment thread Cargo.toml Outdated
@tarcieri
Copy link
Copy Markdown
Contributor

@fjarri can you take a look at this one? Unfortunately it's more breaking changes that block upgrading everything else

@baloo baloo force-pushed the baloo/push-zlqkpkvxqksw branch from 7e75f6e to 0101887 Compare January 20, 2026 18:35
Comment thread Cargo.toml
harness = false

[patch.crates-io]
rand = { git = "https://github.com/rust-random/rand.git" }
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is it still necessary?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sadly yes. This is only used to run tests though. There was some amount of API breakage between rand_core rc-3 and rc-5.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, there hasn't been a new rand release yet. But, since it's a dev-dependency, it won't block publishing a new prerelease

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Could you pin the revision, just so that it doesn't break stuff in future runs?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sidebar: that's what Cargo.lock is for! 😜

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I admit I am not entirely sure if it bumps the revision automatically when the new one is available.

Copy link
Copy Markdown
Contributor Author

@baloo baloo Jan 20, 2026

Choose a reason for hiding this comment

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

I don't believe any dependency management robot supports that.

@fjarri fjarri merged commit e873d92 into entropyxyz:master Jan 20, 2026
12 checks passed
@fjarri
Copy link
Copy Markdown
Collaborator

fjarri commented Jan 20, 2026

Do you need a new release with this?

@tarcieri
Copy link
Copy Markdown
Contributor

Yes that'd be great, thanks!

@baloo baloo deleted the baloo/push-zlqkpkvxqksw branch January 21, 2026 03:40
takumi-earth pushed a commit to earthlings-dev/crypto-primes that referenced this pull request Jan 27, 2026
* chore(deps): bump `rand_core` to `0.10.0-rc-5`

* chore(deps): bump `crypto-bigint` to `0.7.0-pre.21`
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.

3 participants