Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Clarified npm link docs #3075

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

bobrik commented Jan 18, 2013

That wasn't clear for me that node-redis contains package.json where package name is "redis", not "node-redis".

So I made easier example and added note that package-name lives in package.json file. Hope that will save somebody 10 minutes or so.

troy-mc-clure-1024x768

@bobrik bobrik Clarified npm link docs
That wasn't clear for me that node-redis contains `package.json`
where package name is "redis", not "node-redis".

So I made easier example and added note that `package-name` lives
in `package.json` file. Hope that will save somebody 10 minutes or so.
bb65fe0

I think the note above is good, but changing these actually makes things less clear. It's important to show how the directory and package name differ, and how the directory created under node_modules may have a different name from the original directory you're linking from.

Member

domenic commented May 11, 2013

Added that paragraph (but not the renaming of folders) as 441b785.

@domenic domenic closed this May 11, 2013

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