Skip to content

Panda requirement #31

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Panda requirement #31

wants to merge 4 commits into from

Conversation

rubyyan
Copy link

@rubyyan rubyyan commented Jul 17, 2013

Added tests and to_s instance method modified.

@jwo
Copy link
Member

jwo commented Jul 17, 2013

Looks good for panda -- any questions?

@rubyyan
Copy link
Author

rubyyan commented Jul 17, 2013

Hi,

For tiger assignment: "If a player busts (goes over 21), the game should #standfor the player": I understand if a player goes over 21, the player continues playing, is that correct?

Thanks

Date: Tue, 16 Jul 2013 18:27:11 -0700
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [Episode1-Summer2012] Panda requirement (#31)

Looks good for panda -- any questions?


Reply to this email directly or view it on GitHub.

@jwo
Copy link
Member

jwo commented Jul 17, 2013

Hi yan!

In blackjack, the goal is to get as close to 21 without going over. If you do go over, the game stops and you lost.

On Tue, Jul 16, 2013 at 11:27 PM, Yan Go [email protected] wrote:

Hi,
For tiger assignment: "If a player busts (goes over 21), the game should #standfor the player": I understand if a player goes over 21, the player continues playing, is that correct?
Thanks
Date: Tue, 16 Jul 2013 18:27:11 -0700
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [Episode1-Summer2012] Panda requirement (#31)
Looks good for panda -- any questions?

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub:
#31 (comment)

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.

2 participants