search

coreh-deprecated / nide

Beautiful IDE for Node.js
http://coreh.github.com/nide/
713 stars 111 forks source link

Use dive instead of findit to prevent issue #12

Closed pvorb closed 12 years ago

pvorb commented 12 years ago

There is an issue in findit/seq that causes nide to crash when loading a big project:

massimiliano-giroldi asked me, if dive could do the job. I looked at the code of seq and assumed the error got something to do with the chaining of seq.

So I patched nide. Now

Feel free to test it.

coreh commented 12 years ago

Wow man, thank you so much for this!

I tried it on a directory structure with thousands of files and hundreds of subdirectories, and the issue seems to be gone. Project loading also feels so much faster now. Thanks to you and @massimiliano-giroldi.

pvorb commented 12 years ago

NP. That was fun ;).

Cool project, by the way!

max-giro commented 12 years ago

nide & dive rocks together!

P.S.: I think now there is a .nide dir (5.5MB) installed with the module (needs cleaning?).

coreh commented 12 years ago

oops. But that's strange. It's fixed now, but if you look at my .npmignore file, .nide is there.

-- Marco Aurlio

On Thu, Sep 29, 2011 at 11:46 PM, massimiliano-giroldi < reply@reply.github.com>wrote:

nide & dive rocks together!

P.S.: I think now there is a .nide dir (5.5MB) installed with the module (needs cleaning?).

Reply to this email directly or view it on GitHub: https://github.com/Coreh/nide/pull/12#issuecomment-2245762

max-giro commented 12 years ago

Reinstalei de novo e agora está mesmo sem a 'bagagem' toda : )