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

Please publish to npm #4

Closed
NickHeiner opened this issue Jul 2, 2014 · 11 comments
Closed

Please publish to npm #4

NickHeiner opened this issue Jul 2, 2014 · 11 comments

Comments

@NickHeiner
Copy link

npm is a popular package manager for js. If you publish this module there, it will allow it to be consumed via browserify. All you need to do is add a name and version to package.json and run npm publish in this repo.

marsch added a commit to marsch/ui-router.stateHelper that referenced this issue Aug 10, 2014
in order to make this installable by npm [fixes marklagendijk#4]
@marklagendijk
Copy link
Owner

It kind of feels strange for me to publish a browser module to NPM. What is the current trend in this? Is 'everyone' doing this?

@zivc
Copy link

zivc commented Oct 1, 2014

Don't do it, bower is fine.

@NickHeiner
Copy link
Author

It kind of feels strange for me to publish a browser module to NPM.

It's not strange at all! There's no reason that npm needs to only contain node modules. Almost everything on npm is a browser module anyway by the magic of browserify.

Is 'everyone' doing this?

Yes, pretty much. Angular itself is working towards publishing on npm.

Don't do it, bower is fine.

I can't tell if you're :trollface: ing haha. Bower is not fine for many use cases. Why would you insist that everyone use the same package manager?

@zivc
Copy link

zivc commented Oct 2, 2014

Why would you insist that everyone use the same package manager?

From the guy who wants everything to live inside npm and then uses :trollface:

Bower is far from perfect, same with npm. Just when you do it this way you're forcing people into a style and process they may not be willing to adopt. Bower just puts things in a far looser set up so people don't have to spend half a day changing maven, grunt or bash etc.

Best thing would be to do both 👻

@NickHeiner
Copy link
Author

Bower is far from perfect, same with npm.

Totally agree.

Best thing would be to do both

Absolutely! Nothing about supporting one of these package managers precludes supporting the other.

@marsch
Copy link

marsch commented Oct 11, 2014

@zivc both? hehe - lately i got the joke - it's like tabs and spaces right ? :)

@MaxSvargal
Copy link

immature insult removed

@marsch
Copy link

marsch commented Dec 16, 2014

@MaxSvargal +1

@zivc
Copy link

zivc commented Dec 16, 2014

@MaxSvargal mature and welcoming.

@jgoux
Copy link

jgoux commented Feb 13, 2015

I'd love to see this package on NPM. As Angular is published on NPM too, I'd prefer to keep a single package manager instead of installing bower just for one lib.
So a big +1 for it. :)

@kburson
Copy link

kburson commented Feb 22, 2015

+1. Working with angular all my dependencies come thru npm. I used to use bower, but we had so many problems with it that we eventually only used it for setting up our project,then we committed all the packages to our project repository so they would be available for every build rather than using bower to download them on every build. (The build used to break randomly when bower had a panic attack).

I any case, the angular team has migrated to using npm over bower; angular-ui-router is available on npm now, so it makes sense to have the ui-router.stateHelper available on npm as well to keep all dependencies together.

Currently I am forced to use bower to download stateHelper, which then downloads copies of angular and angular-ui-router from bower. so now I have 2 copies of the angular dependencies, one in bower_components and one in node_modules... not very efficient.

Please update the package.json and run npm publish... thank you.

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

Successfully merging a pull request may close this issue.

7 participants