Skip to content

Conversation

@ddaugherty97
Copy link

This change is Reviewable

@kailevy
Copy link

kailevy commented Apr 1, 2016

Reviewed 15 of 15 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions.


chicken.py, line 34 [r1] (raw file):
Include descriptions of each input argument in docstrings


chicken.py, line 45 [r1] (raw file):
Could be random.randint(0,1)


chicken.py, line 205 [r1] (raw file):
Try to avoid magic numbers like these whenever possible


ProjectWriteup.pdf, line 0 [r1] (raw file):
Really nice looking and fun game


Comments from the review on Reviewable.io

@rdiverdi
Copy link

rdiverdi commented May 5, 2016

Review status: 14 of 15 files reviewed at latest revision, 5 unresolved discussions.


chicken.py, line 12 [r2] (raw file):
Nice revision, that's a bit strange, but a pretty cool game mechanic. Thanks for all of the added documentation.


Comments from Reviewable

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