Skip to content
This repository has been archived by the owner on May 25, 2019. It is now read-only.

Move underscore.string from devDependency to dependency #562

Merged
merged 1 commit into from
Jun 29, 2015
Merged

Move underscore.string from devDependency to dependency #562

merged 1 commit into from
Jun 29, 2015

Conversation

martnu
Copy link
Contributor

@martnu martnu commented Jun 28, 2015

To avoid getting

$ yo webapp
module.js:338
    throw err;
          ^
Error: Cannot find module 'underscore.string'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/Users/mart/.nvm/versions/node/v0.12.4/lib/node_modules/generator-webapp/app/index.js:8:10)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)

To avoid getting

```bash
$ yo webapp
module.js:338
    throw err;
          ^
Error: Cannot find module 'underscore.string'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/Users/mart/.nvm/versions/node/v0.12.4/lib/node_modules/generator-webapp/app/index.js:8:10)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
```
silvenon added a commit that referenced this pull request Jun 29, 2015
Move underscore.string from devDependency to dependency
@silvenon silvenon merged commit df96d62 into yeoman:master Jun 29, 2015
@silvenon
Copy link
Member

Nice catch, thanks! 👍

@metakermit
Copy link

Thanks. Can we maybe get a new version bump with this fix included, please? For a temp. fix see here.

@silvenon
Copy link
Member

Released in v0.6.2.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants