lawrence1223/few-plato

Use few.files

Opened this issue · 12 comments

Look at replacing 'app/*/.js' with few.files in case people have a different project structure.

how do you test your changes in local? For example, I changed the code from app/**/.js to app/**/.html

Do I need to publish every time I made changes ?

And I have this error also
screen shot 2015-09-15 at 11 31 59 pm

I have memory leak somewhere?

This does not mean you have a memory leak, but it means you are either try to load a really large file or parse a lot of JSON etc. Do you need to publish if you want people to be able to npm install it remember to follow semver.org. You test the way you would expect by launching the app and looking. If you cannot figure it out, I would be happy to pair with you.

so there is no way to test my changes without publish it to npm

Aha! symlink. thanks

When I symlink or copy the whole folder into node_modules.

I have this error

screen shot 2015-09-16 at 10 57 59 am

after I remove the symlink and install few-plato back, it works.

This would happen if it is symlinked, but not if you copy. Are you sure you ran npm install?

I need to run npm install after I copy the directory to node_modules?

I must did something wrong. I copy the directory to node_modules and it worked

But I still could not figure out what is causing the error about large file or a lot of json.

#2 (comment)

I found the problem.
screen shot 2015-09-17 at 3 30 53 pm

I concatenates the array with these files.
The cause is .concat( few.exclude( 'libraries' ) ).

It seems even I exclude the files in this property, plato still tried to analyze them. And we have a lot a dependencies in the library. So the error occurs.

It works fine if I removed .concat( few.exclude( 'libraries' ) ).