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

Refactor Randomizer #27

Open

3 tasks

Serneum opened this issue Nov 8, 2016 · 0 comments

Member

Serneum commented Nov 8, 2016 •

edited

Loading

See @ericworkman's comment

Rename randomizer
Make the tests great again
Refactor randomizer cause it ain't too pretty (and namespace it better)

In short, this randomizer uses the current year and day of year to as the index of the
list of availablegames to reliably choose the same game for a given day and game list.
I'm not happy with the tests, as I'm not too sure about how to set this up to be tested
reliably (but I barrelled through with it anyway). Paired with @Serneum while doing this.

The text was updated successfully, but these errors were encountered:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment