Skip to content
New issue

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

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

Already on GitHub? # to your account

Rename gen_iter → random_iter, misc.. #1500

Merged
merged 9 commits into from
Oct 1, 2024
Merged

Conversation

dhardy
Copy link
Member

@dhardy dhardy commented Sep 28, 2024

  • Added a CHANGELOG.md entry

This extracts the non-inherent-methods stuff from #1492.

@dhardy dhardy requested a review from newpavlov September 28, 2024 08:36
Copy link
Member

@newpavlov newpavlov left a comment

Choose a reason for hiding this comment

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

Maybe it's also worth to rename gen_range to random_range?

@dhardy
Copy link
Member Author

dhardy commented Sep 28, 2024

Maybe... but it's longer and a significant breaking change.

@dhardy dhardy merged commit 66b11eb into rust-random:master Oct 1, 2024
15 checks passed
@dhardy dhardy deleted the rng branch October 1, 2024 13:59
benjamin-lieser pushed a commit to benjamin-lieser/rand that referenced this pull request Feb 5, 2025
This extracts the non-inherent-methods stuff from rust-random#1492.
benjamin-lieser pushed a commit to benjamin-lieser/rand that referenced this pull request Feb 5, 2025
This extracts the non-inherent-methods stuff from rust-random#1492.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants