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

Rename Random.choice() to Random.pick() #181

Merged
merged 1 commit into from
Feb 20, 2022
Merged

Conversation

Xrayez
Copy link
Contributor

@Xrayez Xrayez commented Feb 20, 2022

Part of #176.

  1. To avoid name collision with Random.choices() for weighed selection.
  2. Random.pick() will go along with Random.pop()
  3. Faster and easier to type.

1. To avoid name collision with `Random.choices()` for weighed selection.
2. `Random.pick()` will go along with `Random.pop()`
3. Faster and easier to type.
@Xrayez Xrayez added enhancement Improvement of existing features breaks compat Compatibility breakage topic:core labels Feb 20, 2022
@Xrayez Xrayez added this to the 2.0-gd3 milestone Feb 20, 2022
@Xrayez Xrayez merged commit f0018df into gd3 Feb 20, 2022
@Xrayez Xrayez deleted the rename-choice-to-pick branch February 20, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaks compat Compatibility breakage enhancement Improvement of existing features topic:core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant