filewatcher and new files #29

Closed
Raynos opened this Issue Jul 7, 2012 · 10 comments

Projects

None yet

3 participants

@Raynos
Contributor
Raynos commented Jul 7, 2012

It doesn't handle watching new files in an elegant fashion.

Ideally we want to add a watcher to each folder, detect new files added and then add any new files to the watcher

@airportyh
Collaborator

This is made harder because of glob support. We might have to either side step glob support in some way, or to dive into the glob code.

@Raynos
Contributor
Raynos commented Jul 8, 2012

I personally think the optimum solution is to allow watching on files, folders & globs.

Then for folders we special case it as a recursive watcher on the folder and any sub folders / files including the new file detection.

This allows me to use either test/**/*.js or test

@airportyh
Collaborator

@Raynos resolved this with a new file watcher I wrote https://github.com/airportyh/fireworm. Please try it out - create new test files and it should automatically pick them up too.

@airportyh airportyh closed this Nov 24, 2012
@airportyh airportyh reopened this Nov 28, 2012
@szkrd
szkrd commented Jan 4, 2013

Maybe it's just me, but after downloading testem and doing the hello world example it still does not handle new files, only the ones that had been already created before launching the framework (I'm on Windows 7, using testem 0.2.50 and node 0.8.6).

@airportyh
Collaborator

This is not working yet because I had to back out the change to use the new file watcher, which is why it was reopened. Sorry for the confusion.

@szkrd
szkrd commented Jan 4, 2013

I see, no problem and thank you for the answer.

@Raynos
Contributor
Raynos commented Jan 6, 2013

The fact that this still doesn't work really kills my work flow.

@airportyh
Collaborator

This is done! 3458c5f Hope it stays done :)

@airportyh airportyh closed this Aug 22, 2013
@Raynos
Contributor
Raynos commented Aug 22, 2013

yes! thanks :)

@airportyh
Collaborator

Also checkout https://github.com/airportyh/watchem if you want something smaller/simpler.

@Raynos Raynos referenced this issue in airportyh/fireworm Dec 22, 2012
Closed

Capture new files #1

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