Closed jacooobi closed 8 years ago
Thanks @fhwang for your suggestions, I've modified the code.
Other then the CC issues, LGTM
What are the CC issues? I can't access that page.
@jacobi007 Hm when I click on the link to CC issues, I don't see any new ones:
So I think we can disregard those for now.
Anything else blocking merging this?
cc @codeclimate/review
Yeah, the code climate issues thing is a bug on our end. I've seen it before. :cry:
Looks good based from some testing.
As listed in spec