Skip to content

Conversation

@yordanosd
Copy link

Thanks!

@sudocrystal
Copy link

Where's your README? I can't see your thought process~

overall_mood_result = {}
overall_mood_result[":-)"] = 0
overall_mood_result[":-|"] = 0
overall_mood_result[":-("] = 0

Choose a reason for hiding this comment

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

If you do overall_mood_result = Hash.new(0) then you don't need to these three lines initing the key/values....

@sudocrystal
Copy link

You made lots of good choices in structure. Good job!

@yordanosd
Copy link
Author

Thanks for the feedback.

I thought the ReadMe file was just an in class exercise so I didn't finish
it and I didn't think we had to include it in the pull request. Sorry, I
just pushed a completed ReadMe file to my repo. Let me know if I need to
push it or anything.

Thanks for your help.

Y

On Sat, Apr 2, 2016 at 2:59 PM, Crystal notifications@github.com wrote:

You made lots of good choices in structure. Good job!


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#10 (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