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 Array.swap_elements and various Random methods (RFC 46). #2128

Merged
merged 3 commits into from
Sep 20, 2017

Conversation

jemc
Copy link
Member

@jemc jemc commented Aug 3, 2017

Resolves #2127.

@jemc jemc changed the title Add Array.swap and various Random methods (RFC 46). Add Array.swap_elements and various Random methods (RFC 46). Aug 3, 2017
@jemc jemc force-pushed the feature/random-methods branch from 2ca86f8 to 61c8fc8 Compare August 3, 2017 02:37
@SeanTAllen
Copy link
Member

Add a test for "swap elements"?

@SeanTAllen
Copy link
Member

I'm not sure that the usual CHANGELOG tag will properly capture this change. Manual CHANGELOG entries to cover the different additions?

@mfelsche
Copy link
Contributor

i took the freedom to add the requested test.

@mfelsche mfelsche force-pushed the feature/random-methods branch from 30a6313 to ff56d02 Compare September 19, 2017 10:33
@mfelsche
Copy link
Contributor

i extended my claimed freedom to rebase and also add a manual changelog entry.
Let's get that RFC rolling.

@jemc
Copy link
Member Author

jemc commented Sep 19, 2017

Restarting the OSX Travis builds that failed to see if they were a fluke...

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