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

Ditch Gulp entirely #3

Closed
Silic0nS0ldier opened this issue Feb 10, 2018 · 1 comment
Closed

Ditch Gulp entirely #3

Silic0nS0ldier opened this issue Feb 10, 2018 · 1 comment

Comments

@Silic0nS0ldier
Copy link
Member

While working on #2 I've noticed that this plugin doesn't work like existing Gulp plugins, much to its own detriment.

  1. The configuration file is provided via gulp.src, only to have to be read again (path is pulled from the Vinyl object, then require is used).
  2. gulp.src is used internally, per bundle. If a file is included in multiple bundles, it'll need to be read several times.
  3. How the plugin works stunts any efforts to further stabilize and optimize builds in UserFrosting. If the plugin works on files exclusively in the pipeline, then it will be possibly to revise the build process to cut bundling time in half while greatly reducing IO.
@Silic0nS0ldier
Copy link
Member Author

See #6

@Silic0nS0ldier Silic0nS0ldier removed this from the 3.0.0 milestone Apr 30, 2018
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

No branches or pull requests

1 participant