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

TypeError: Cannot read property 'bold' of undefined #7

Closed
collinforrester opened this Issue Sep 9, 2013 · 13 comments

Comments

Projects
None yet
5 participants
@collinforrester
Contributor

collinforrester commented Sep 9, 2013

TypeError: Cannot read property 'bold' of undefined
    at Object.<anonymous> (c:\Users\Collin\NODE\node_modules\generator-gruntplugin\node_modules\yeoman-generator\lib\util\common.js:5:56)

This happened to my other generators as well, but basically yeoman-generator needs upgraded to work with the newer versions of yo since they use chalk.js now to do the colors. I'm sure you guys know this though as you're probably the ones who made the change. Thanks!

I'll submit a PR if I have time this week.

@passy

This comment has been minimized.

Show comment
Hide comment
@passy

passy Sep 9, 2013

Member

This should have been fixed with 0.0.5. Are you sure you're on the latest
version?

Pascal Hartig
Software Engineer

http://passy.me/
Twitter: https://twitter.com/passy
Google+: http://gplus.to/passy

On Mon, Sep 9, 2013 at 3:53 PM, collinforrester notifications@github.comwrote:

TypeError: Cannot read property 'bold' of undefined
at Object. (c:\Users\IGEN721\NODE\node_modules\generator-gruntplugin\node_modules\yeoman-generator\lib\util\common.js:5:56)

This happened to my other generators as well, but basically
yeoman-generator needs upgraded to work with the newer versions of yo since
they use chalk.js now to do the colors. I'm sure you guys know this though
as you're probably the ones who made the change. Thanks!

I'll submit a PR if I have time this week.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7
.

Member

passy commented Sep 9, 2013

This should have been fixed with 0.0.5. Are you sure you're on the latest
version?

Pascal Hartig
Software Engineer

http://passy.me/
Twitter: https://twitter.com/passy
Google+: http://gplus.to/passy

On Mon, Sep 9, 2013 at 3:53 PM, collinforrester notifications@github.comwrote:

TypeError: Cannot read property 'bold' of undefined
at Object. (c:\Users\IGEN721\NODE\node_modules\generator-gruntplugin\node_modules\yeoman-generator\lib\util\common.js:5:56)

This happened to my other generators as well, but basically
yeoman-generator needs upgraded to work with the newer versions of yo since
they use chalk.js now to do the colors. I'm sure you guys know this though
as you're probably the ones who made the change. Thanks!

I'll submit a PR if I have time this week.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7
.

@collinforrester

This comment has been minimized.

Show comment
Hide comment
@collinforrester

collinforrester Sep 10, 2013

Contributor

0.0.5 of generator-gruntplugin? I can only get (and see in NPM) up to 0.0.3 and in 0.0.3 the dependency for yeoman-generator still uses the old welcome message format.

Contributor

collinforrester commented Sep 10, 2013

0.0.5 of generator-gruntplugin? I can only get (and see in NPM) up to 0.0.3 and in 0.0.3 the dependency for yeoman-generator still uses the old welcome message format.

@addyosmani

This comment has been minimized.

Show comment
Hide comment
@addyosmani

addyosmani Oct 1, 2013

Member

Confirmed this is still an issue today:

 yo gruntplugin

TypeError: Cannot read property 'bold' of undefined
    at Object.<anonymous> (/Users/addyo/projects/node_modules/generator-gruntplugin/node_modules/yeoman-generator/lib/util/common.js:5:56)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/Users/addyo/projects/node_modules/generator-gruntplugin/node_modules/yeoman-generator/lib/base.js:91:26)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
Member

addyosmani commented Oct 1, 2013

Confirmed this is still an issue today:

 yo gruntplugin

TypeError: Cannot read property 'bold' of undefined
    at Object.<anonymous> (/Users/addyo/projects/node_modules/generator-gruntplugin/node_modules/yeoman-generator/lib/util/common.js:5:56)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/Users/addyo/projects/node_modules/generator-gruntplugin/node_modules/yeoman-generator/lib/base.js:91:26)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
@collinforrester

This comment has been minimized.

Show comment
Hide comment
@collinforrester

collinforrester Oct 1, 2013

Contributor

I think it's because the upped version numbers that I committed haven't actually been included in a version published to NPM yet.

When I install from NPM I get that issue, but when I install using the higher yeoman-generator version, it works fine.

Contributor

collinforrester commented Oct 1, 2013

I think it's because the upped version numbers that I committed haven't actually been included in a version published to NPM yet.

When I install from NPM I get that issue, but when I install using the higher yeoman-generator version, it works fine.

@addyosmani

This comment has been minimized.

Show comment
Hide comment
@addyosmani

addyosmani Oct 2, 2013

Member

I pushed a new version to this repo yesterday. Could you try installing from master and let me know if it works okay for you?

Member

addyosmani commented Oct 2, 2013

I pushed a new version to this repo yesterday. Could you try installing from master and let me know if it works okay for you?

@collinforrester

This comment has been minimized.

Show comment
Hide comment
@collinforrester

collinforrester Oct 2, 2013

Contributor

It works fine when I install from master.

Contributor

collinforrester commented Oct 2, 2013

It works fine when I install from master.

@addyosmani

This comment has been minimized.

Show comment
Hide comment
@addyosmani

addyosmani Oct 2, 2013

Member

Fantastic. I'm just waiting on one more review and we'll publish to npm.

Member

addyosmani commented Oct 2, 2013

Fantastic. I'm just waiting on one more review and we'll publish to npm.

@addyosmani addyosmani closed this Oct 2, 2013

@stryju

This comment has been minimized.

Show comment
Hide comment
@stryju

stryju Oct 30, 2013

issue still exists, since the last npm-published version is 0.0.3

stryju commented Oct 30, 2013

issue still exists, since the last npm-published version is 0.0.3

@collinforrester

This comment has been minimized.

Show comment
Hide comment
@collinforrester

collinforrester Oct 30, 2013

Contributor

@stryju I don't believe they have released it yet. Try installing from master for now and it should work.

I think npm install git://github.com/yeoman/generator-gruntplugin.git is all it takes.

Contributor

collinforrester commented Oct 30, 2013

@stryju I don't believe they have released it yet. Try installing from master for now and it should work.

I think npm install git://github.com/yeoman/generator-gruntplugin.git is all it takes.

@stryju

This comment has been minimized.

Show comment
Hide comment
@stryju

stryju Oct 30, 2013

yep, I know

just wanted to "refresh" the issue, since following the "normal" installation steps will lead to this error

stryju commented Oct 30, 2013

yep, I know

just wanted to "refresh" the issue, since following the "normal" installation steps will lead to this error

@ericf

This comment has been minimized.

Show comment
Hide comment
@ericf

ericf Nov 17, 2013

Seems like this should be published to npm :-/

ericf commented Nov 17, 2013

Seems like this should be published to npm :-/

@passy

This comment has been minimized.

Show comment
Hide comment
@passy

passy Nov 17, 2013

Member

Pushed v0.0.5 to npm.

Pascal Hartig
Software Engineer

http://passy.me/
Twitter: https://twitter.com/passy
Google+: http://gplus.to/passy

On Sun, Nov 17, 2013 at 12:57 AM, Eric Ferraiuolo
notifications@github.comwrote:

Seems like this should be published to npm :-/


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-28639970
.

Member

passy commented Nov 17, 2013

Pushed v0.0.5 to npm.

Pascal Hartig
Software Engineer

http://passy.me/
Twitter: https://twitter.com/passy
Google+: http://gplus.to/passy

On Sun, Nov 17, 2013 at 12:57 AM, Eric Ferraiuolo
notifications@github.comwrote:

Seems like this should be published to npm :-/


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-28639970
.

@addyosmani

This comment has been minimized.

Show comment
Hide comment
@addyosmani

addyosmani Nov 17, 2013

Member

Thanks @passy!

Member

addyosmani commented Nov 17, 2013

Thanks @passy!

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