Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Add hint about bower installation failure #7908

Closed
wants to merge 1 commit into from
Closed

Add hint about bower installation failure #7908

wants to merge 1 commit into from

Conversation

tinne
Copy link

@tinne tinne commented Jun 19, 2014

When running npm install, this failed for me due to the reasons mentioned in the StackOverflow article cited. Creating the config file worked for me, did not know where to apply the better ranked config option. Feel free to add that if you know better.

When running npm install, this failed for me due to the reasons mentioned in the StackOverflow article cited. Creating the config file worked for me, did not know where to apply the better ranked config option. Feel free to add that if you know better.
@mary-poppins
Copy link

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

@tinne tinne added cla: no and removed cla: yes labels Jun 19, 2014
@rodyhaddad rodyhaddad added this to the 1.3.0 milestone Jun 23, 2014
@rodyhaddad
Copy link
Contributor

Shouldn't we be using --config.interactive=false and fixing this instead of adding a note to the docs?

@rodyhaddad
Copy link
Contributor

Hm, we invoke bower programmatically, and in the docs of bower, they say

When using bower programmatically, prompting is disabled by default

So I'm not sure this is actually an issue with angular.

@rodyhaddad
Copy link
Contributor

cc @petebacondarwin

@petebacondarwin
Copy link
Contributor

Woh! Stack Overflow is down for routine maintenance!

@petebacondarwin
Copy link
Contributor

@rodyhaddad - this issue is about the phonecat tutorial so we are not using the programmatic method that you link (that is only for building angular itself).

We already have a .bowerrc in phonecat so I suggest that we just add this to it:

{
  "interactive": false
}

@petebacondarwin
Copy link
Contributor

@tinne - thanks for highlighting this.

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

Successfully merging this pull request may close these issues.

4 participants