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

Panda and Tiger Level #17

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

Panda and Tiger Level #17

wants to merge 2 commits into from

Conversation

tmaffia
Copy link

@tmaffia tmaffia commented Nov 24, 2013

I had difficulty understanding the point of the FoodBarge class. Of course I understand that this is just an abstraction for the purpose of learning, but since I am new to Ruby its hard to say if the class I wrote and the other I modified solved the problem.

@jwo
Copy link
Member

jwo commented Nov 25, 2013

Code looks great, nice tests as well.

The food barge could be extended to have inventory -- like an actual place with actual inventory of food. That's the idea if we go much further.

end

end

class Human
include Animal
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should check your browser's settings --- this looks like two tabs instead of 2 spaces.

(2 spaces are the preferred way in Ruby).

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