any reason why variable name in README.md is not in camelCase? about generator-node

[hemanth]

No it's not a bug, look at index.js#L45

[stevemao]

So this is not using camelCase on purpose? Sorry I don't really understand this.

[hemanth]

@stevemao Are you facing any issue? That's just a naming convention.

[stevemao]

Not really, just think variable names should be camelCase. The ones in the code are camelCased but not in README.md. I have to manually change it every time.

[stevemao]

https://github.com/yeoman/generator-node/blob/master/app/templates/cli.js#L4

This is using safeSlugname

But in README.md it's

var <%= slugname %> = require('<%= slugname %>');

[stevemao]

I knew there was a problem 😸

[hemanth]

heh heh, we must use _.slugify rather than manually slugifying it.

[hemanth]

Ha, we are L46

[stevemao]

Looks like safeSlugname is camelCased name.

[hemanth]

Yes, that was done here as a fix, but I still feel that the readme must be slugname not safe slug.

[stevemao]

Any reason for this?

[hemanth]

Readme must not be safe, but rather must be plain slug.

Say if your module is named as is-mp3 doing a var is-mp3 is not a vaild literal, so we had to use safe names, that would result in isMp3 hence we could do var isMp3 = require('is-mp3')

[stevemao]

Isn't the part in readme sample code?

[hemanth]

@stevemao Sorry for the ping-pong the readme template must be:

var <%= safeSlugname %> = require('<%= slugname %>');

Feel free to send a PR.

[stevemao]

Sure 😃