Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add recipe for fast browserify builds (without watchify) #672

Closed
wants to merge 2 commits into from
Closed

Add recipe for fast browserify builds (without watchify) #672

wants to merge 2 commits into from

Conversation

marcelbeumer
Copy link

By not using watchify you prevent having multiple watchers watch the same file (gulp uses gaze and watchify uses chokidar), and it seems easy enough to implement the caching ourselves.

@lucasmotta
Copy link

Works like a charm, well done @marcelbeumer :)

@yocontra
Copy link
Member

yocontra commented Dec 9, 2014

Seems like this should be broken out into a module. Also watchify does more than just watch */.js - it actually walks the dependency tree and only watches what is being used.

I'm still in favor of watchify since it handles all of this under the hood - this seems like too much code for a simple task.

@yocontra
Copy link
Member

I just updated the watchify recipe, I think it solves this problem just fine. This PR basically rewrites what watchify is doing under the hood - I would rather people use the module that makes it easier

@yocontra yocontra closed this Dec 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants