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

install yeoman from NPM in install script. #315

Merged
merged 2 commits into from Sep 10, 2012

Conversation

Projects
None yet
3 participants
Member

paulirish commented Sep 10, 2012

Now that Yeoman is in NPM we'll install directly from there.

Look good, @tomlane ?

ref #298

Owner

sindresorhus commented Sep 10, 2012

lgtm

Member

paulirish commented Sep 10, 2012

Thx.

Unrelated ; Can you sudo the rm in the ununstall?

Owner

sindresorhus commented Sep 10, 2012

Done

Contributor

tomlane commented Sep 10, 2012

Looks good here :)

@sindresorhus sindresorhus added a commit that referenced this pull request Sep 10, 2012

@sindresorhus sindresorhus Merge pull request #315 from yeoman/installfromnpm
install yeoman from NPM in install script.
307fc66

@sindresorhus sindresorhus merged commit 307fc66 into master Sep 10, 2012

@paulirish paulirish added a commit that referenced this pull request Apr 24, 2015

@paulirish paulirish strip this nasty SHA version. thx #315 a64e841

@sindresorhus sindresorhus added a commit that referenced this pull request Apr 24, 2015

@sindresorhus sindresorhus Merge pull request #315 from yeoman/installfromnpm
install yeoman from NPM in install script.
fe7cefd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment