Added a new article. #71

Closed
wants to merge 4 commits into
from

2 participants

@hemanth

Hello Creationix,

Have added a new article called "Hitchhikers guide to npm" hope it has meet the article standards 👨
Thank you!

@creationix
Owner

Sorry I missed this. My only complaint is you advocate installing node and npm using a custom script. That's a pretty big security issue because you could change the script to do evil things. And even if you don't change it and you're honest, your server becomes a target for hackers. It's not hard to intercept http traffic and change it. I don't feel comfortable promoting that. You could inline the ~10 lines of your script in the article instead maybe?

@hemanth

@creationix Wow! That script is so out dated now.

I think 2 years ago it made sense, but now node itself installs npm also.

I will remove that and link to this.

@creationix
Owner

Much better. If you still want me to publish this, update the date in metadata and let me know.

And sorry for missing the original PR.

@hemanth

@creationix Well...the remaining is still relevant, ok will update the meta data.

And it's OK that this went unnoticed!

@hemanth

@creationix I'm feeling like we should ditch this? Or mostly I need to re-write the whole thing now heh heh. npm --save was not there then.

@creationix
Owner

Ok, just let me know if/when you want to publish something. If I don't respond within a couple hours, feel free to ping me again here or on twitter.

@hemanth

Thanks! 👍 Will close this as of now.

@hemanth hemanth closed this Feb 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment