Skip to content

use now allowed randbytes and other helpers #19838

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 21, 2025
Merged

use now allowed randbytes and other helpers #19838

merged 1 commit into from
Jul 21, 2025

Conversation

altendky
Copy link
Contributor

@altendky altendky commented Jul 16, 2025 •
edited
Loading

Purpose:

https://docs.python.org/3.13/library/random.html#random.randbytes

Added in version 3.9.

Current Behavior:

New Behavior:

Testing Notes:

@altendky altendky added Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes labels Jul 16, 2025
@altendky altendky requested a review from a team as a code owner July 16, 2025 21:28
@github-actions GitHub Actions
Copy link
Contributor

File Coverage Missing Lines
chia/_tests/core/test_merkle_set.py 0.0% lines 282
Total Missing Coverage
12 lines Unknown 91%

@altendky altendky requested a review from emlowe July 17, 2025 16:50
@Starttoaster Starttoaster merged commit 8bba601 into main Jul 21, 2025
520 of 523 checks passed
@Starttoaster Starttoaster deleted the randbytes branch July 21, 2025 15:55
@altendky altendky mentioned this pull request Jul 21, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@emlowe emlowe emlowe approved these changes

Assignees

No one assigned

Labels

Changed Required label for PR that categorizes merge commit message as "Changed" for changelog Exclude_Notes Use this label if the changes in the PR should be excluded from the release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@altendky @emlowe @Starttoaster