roguerb / pull-request-workshop

0 stars 6 forks source link

multiply is doing add in roge math. #9

Closed RacheleJohnson closed 8 years ago

RacheleJohnson commented 8 years ago

I saw in the roge_math file that it was saying def self.multiply(a, b) a + b. you can not multiply a+b

conroywhitney commented 8 years ago

Hey @RacheleJohnson. Thanks for pointing this out!

Would you please create a new Pull Request that fixes this method, as well as updates the specs so this issue doesn't crop up again in the future?

Thanks!

conroywhitney commented 8 years ago

Closing per PR #19. Thanks @RacheleJohnson! 👍