Skip to content

yeomaninsights.py should be installed on first run of bin/yeoman #192

Closed
ebidel opened this Issue Aug 9, 2012 · 7 comments

2 participants

@ebidel
Yeoman member
ebidel commented Aug 9, 2012

This is in case users use npm to install instead of install.sh.

setup script:
https://github.com/yeoman/yeoman/blob/c5796a7db28c70420e5bbc98ef2742e262e47e8e/setup/install.sh#L197-201

@ebidel ebidel was assigned Aug 9, 2012
@ebidel
Yeoman member
ebidel commented Aug 10, 2012

When are we launching? I don't think I'll have this ready by next Monday.

@paulirish @addyosmani

@addyosmani
Yeoman member

@ebidel I don't think we've nailed down a new date yet. We decided to evaluate how stable the project is before we make a call. I think you should be okay for getting this done sometime next week. No worries!

@ebidel
Yeoman member
ebidel commented Aug 10, 2012

Fellas, how does the npm install -g way of installing going to work in practice? npm won't be able to run python setup.py install --quiet --force --user --install-scripts=~/.yeoman/insight no?

@addyosmani
Yeoman member

@ebidel To the best of my knowledge we wouldn't be able to run it if opting for npm install -g. Perhaps however, if someone were to fire up yeoman for the first time we could check for the existence of insight and kick off the install of it there? Would that work?

@ebidel
Yeoman member
ebidel commented Aug 10, 2012

How do they get the yeomaninsight.py script then? I can figure out a way to pull it down, but this will complicate things.
That check would need to be made every time since yeomaninsight.py could have been updated and the new version needs to be installed.

@ebidel
Yeoman member
ebidel commented Aug 10, 2012

Think I found a way to install the py script from package.json :)

@addyosmani
Yeoman member

that would be awesome :)

@ebidel ebidel added a commit that closed this issue Aug 10, 2012
@ebidel ebidel Fixes #192 8530cb8
@ebidel ebidel closed this in 8530cb8 Aug 10, 2012
@szinya szinya pushed a commit to menthainternet/yeoman that referenced this issue Sep 17, 2014
@ebidel ebidel Fixes #192 20507d4
@ebidel ebidel added a commit that referenced this issue Apr 24, 2015
@ebidel ebidel Fixes #192 11c5c9d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.