Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add post install for bower install in new packages #111

Closed
jwebbdev opened this issue Aug 3, 2015 · 4 comments
Closed

Add post install for bower install in new packages #111

jwebbdev opened this issue Aug 3, 2015 · 4 comments

Comments

@jwebbdev
Copy link
Member

jwebbdev commented Aug 3, 2015

Should we add a block for

  "scripts": {
    "postinstall" : "bower install"
  },

It seems a lot of people don't realize they need to add this once adding bower dependencies to cause bower installs to cascade along with npm.

It would add a little time to general npm installs based on how many packages you have, but I doubt it would be much, and if it's a concern someone can always remove the post install scripts for deployment so life is easier on new devs.

@BrockMcKean
Copy link

Yes. This is related to linnovate/meanio#65 and I think it should be enabled by default. It will have to be to support a complete mean package abstraction.

@liorkesos
Copy link
Member

This would automatically execute a bower install by default for all packages in your view?

@jwebbdev
Copy link
Member Author

jwebbdev commented Aug 4, 2015

Yeah, as the npm install cascades through the packages it would cause the bower install to cascade along with it.

@timelf123
Copy link
Contributor

Fixed by 6be2525

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants