Replaced old-style octal literals #63

Closed
wants to merge 3 commits into
from

Projects

None yet

2 participants

Contributor
M1C commented Apr 23, 2012

Partial fix for #62

Contributor
M1C commented Apr 24, 2012

The 2nd commit wasn't intended, but you can still pull it if you want to :)

Owner
amasad commented Apr 25, 2012

You're right.
Turns out my coffee-script version is 1.1 and the octal and eval thing is not an issue on my version.
It maybe a good idea to add a package.json file with module versions to avoid issues like that in the future. Nonetheless the issues you've mentioned must be fixed.

I'm pulling the first two commits. The problem with the third one is that its catching a general error (that could be anything) and showing a specific message.

Thanks!

@amasad amasad closed this Apr 25, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment