Skip to content

convert Alea PRNG to Swift#26

Merged
ningkaiqiang merged 1 commit intoopen-spaced-repetition:mainfrom
tomtau:swift-alea
Sep 3, 2025
Merged

convert Alea PRNG to Swift#26
ningkaiqiang merged 1 commit intoopen-spaced-repetition:mainfrom
tomtau:swift-alea

Conversation

@tomtau
Copy link
Contributor

@tomtau tomtau commented Sep 2, 2025

to remove the dependency on JavaScriptCore which isn't available on some targets

@ningkaiqiang
Copy link
Collaborator

Amazing, thank uuuu!!!

@tomtau
Copy link
Contributor Author

tomtau commented Sep 2, 2025

BTW @ningkaiqiang do you plan to update the algorithm version to v6?

Copy link
Member

@ishiko732 ishiko732 left a comment

Choose a reason for hiding this comment

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

LGTM

@ningkaiqiang
Copy link
Collaborator

Absolutely, in a few days @tomtau

@L-M-Sherlock
Copy link
Member

Do we have unit tests to make sure that the PR's implementation is consistent with previous one?

@tomtau
Copy link
Contributor Author

tomtau commented Sep 3, 2025

Copy link
Member

@L-M-Sherlock L-M-Sherlock left a comment

Choose a reason for hiding this comment

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

LGTM

@ningkaiqiang ningkaiqiang merged commit d078b64 into open-spaced-repetition:main Sep 3, 2025
1 check passed
@tomtau tomtau deleted the swift-alea branch September 3, 2025 09:23
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.

4 participants