simon-dt / gulp-twig

Twig plugin for gulp.js, The streaming build system. Looking for maintainer or collaborators. See wiki
https://github.com/zimmen/gulp-twig/wiki/Looking-for-maintainer-or-collaborator(s)
MIT License
62 stars 33 forks source link

npm install doesn't install requirements #4

Closed Gaya closed 9 years ago

Gaya commented 9 years ago

gulp-util is not found on first use. Seems like it's in the devDependencies, but it's used in production. Needs to move the dependency to dependencies.

simon-dt commented 9 years ago

Hmm that's no good. You know what gulp-util function is used? There should be no deps on a build tool

On 09 Oct 2014, at 09:28, Gaya Kessler notifications@github.com wrote:

gulp-util is not found on first use. Seems like it's in the devDependencies, but it's used in production. Needs to move the dependency to dependencies.

— Reply to this email directly or view it on GitHub.

Gaya commented 9 years ago

It's used for gutil.PluginError and gutil.log. Quite a few times.

simon-dt commented 9 years ago

In the actual source? That is bad. Gona remove that.

On 09 Oct 2014, at 09:58, Gaya Kessler notifications@github.com wrote:

It's used for gutil.PluginError and gutil.log. Quite a few times.

— Reply to this email directly or view it on GitHub.

simon-dt commented 9 years ago

So I'd rather remove the dep altogether. Perhaps during a less interesting talk today;-)

Gaya commented 9 years ago

Alright! :)

raphaelgoetter commented 9 years ago

Hello, I have the same issue : I just used exactly the same code than this one : https://www.npmjs.org/package/gulp-twig/

And get an error :

Error: Cannot find module 'gulp-util'
    at Function.Module._resolveFilename
    at Function.Module._load (module.js:
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (C:\Users\Raph
twig\index.js:3:13)
    at Module._compile (module.js:456:26
    at Object.Module._extensions..js (mo
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:
    at Module.require (module.js:364:17)

I'm using Version "0.1.0 last updated a month ago"

colynb commented 9 years ago

I see you made the update in github but it doesn't seem to be reflected in npm. I just did an npm install gulp-twig and it shows gulp-util as a dev dependency.