Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add comment to specify the rand version #473

Merged
merged 2 commits into from
Mar 12, 2025

Conversation

Its-Just-Nans
Copy link

@Its-Just-Nans Its-Just-Nans commented Feb 11, 2025

Hello,

This PR is not super useful BUT can be useful.

Of course this PR should be kind of reversed/updated when the rand 0.9 will be supported

Linked to

Feel free to close if not useful enough to be merged

@tarcieri
Copy link
Member

@Its-Just-Nans can you rebase?

@Its-Just-Nans
Copy link
Author

Its-Just-Nans commented Feb 13, 2025

@Its-Just-Nans can you rebase?

Yes, done

Copy link
Member

@dignifiedquire dignifiedquire left a comment

Choose a reason for hiding this comment

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

thanks, it is a bit annoying, but I guess for the time being at least helps users understand the situation

@newpavlov
Copy link
Member

This PR should be probably merged into the 0-9-x branch and released as rsa v0.9.8. I don't think it makes sense to merge it into master in the light of #478.

@Its-Just-Nans Its-Just-Nans changed the base branch from master to 0-9-x February 21, 2025 13:32

Verified

This commit was signed with the committer’s verified signature.
@Its-Just-Nans
Copy link
Author

I've updated the base branch

@newpavlov
Copy link
Member

Hm. Should we merge it regardless of the CI failures?

@tarcieri
Copy link
Member

Not sure what broke the build, but maybe we should fix it

Verified

This commit was signed with the committer’s verified signature.
@Its-Just-Nans
Copy link
Author

Hi

The commit "Fix CI" 3f83141 fixed the CI by removing unused import (also by updating cargo.lock)

@newpavlov newpavlov merged commit bd3575b into RustCrypto:0-9-x Mar 12, 2025
9 checks passed
@tarcieri tarcieri mentioned this pull request Mar 15, 2025
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.

None yet

4 participants