Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

improve the way undefined is used in a few places #620

wants to merge 2 commits into


None yet
2 participants

vikstrous commented Oct 20, 2012

These changes should have no effect on the logic and make the code slightly smaller and simpler. I couldn't run the tests locally. Please test.

Also, any idea why I can't run the tests?

$ rake test
Can't open 'test/runner.coffee'
rake aborted!
Command failed with status (255): [script/test...]

Tasks: TOP => test
(See full trace by running task with --trace)
$ ls test/runner.coffee 

mislav commented Nov 25, 2012

Thanks! Pulled.

You need phantomjs to run the tests. On Max OS X: brew install phantomjs

@mislav mislav closed this Nov 25, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment