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

docs(general): correct bower version separator from '@' to '#' #14819

Closed
wants to merge 1 commit into from

Conversation

guillaumegarcia13
Copy link
Contributor

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Docs update

What is the current behavior? (You can also link to an open issue here)

bower syntax is incorrect, the docs says '@' instead of '#' to specify version

What is the new behavior (if this is a feature change)?

bower syntax is corrected

Does this PR introduce a breaking change?

No (considering it's documentation)

Please check if the PR fulfills these requirements

Other information:

@gkalpak
Copy link
Member

gkalpak commented Jun 23, 2016

Is this new? I'm pretty sure @ used to work too.

@guillaumegarcia13
Copy link
Contributor Author

@gkalpak I am no bower expert but I did try the version in the docs with the given syntax and failed miserably... Took me a couple of minutes to realize the angular docs were in question.

The bower version using '#' seems the way to go now, I don't know about earlier.
https://bower.io/docs/api/#install

I guess it can be confusing since npm is using the different '@' syntax.
https://docs.npmjs.com/cli/install

@gkalpak gkalpak closed this in fb3bbc9 Jun 25, 2016
gkalpak pushed a commit that referenced this pull request Jun 25, 2016
@guillaumegarcia13 guillaumegarcia13 deleted the patch-1 branch June 29, 2016 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants