mozilla / doctorjs

A set of static analysis tools for JavaScript
Other
739 stars 67 forks source link

solved the require errors #39

Closed sergioramos closed 12 years ago

marekkalnik commented 12 years ago

If anyone is still managing this project (seems dead...) I give +1 to this PR because it solves one of the bugs in tests (when using node 0.7.7). But I still didn't succeed I running it, so I can't say there are no bugs introduced by it...

sergioramos commented 12 years ago

The api should work, so the example should also work. Try:

node example.js

The project isn't dead yet:

ramitos: @BrendanEich is doctorjs a dead project? tweet BrendanEich: @ramitos not dead for good, intern about to revive. cc'ing @littlecalculist tweet littlecalculist: @BrendanEich @ramitos Let's just say it's been dormant. We're reaching for the smelling salts starting next week. :) tweet

Also, if you want only the jsctags api you can use this branch ramitos/doctorjs:node

marekkalnik commented 12 years ago

Thanks for the response. Seems like there was a problem with node 0.7.7, I've merged your PR, updated Node to 0.8.1 and the example runs fine.