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

npm support #101

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

npm support #101

wants to merge 2 commits into from

Conversation

maxdow
Copy link

@maxdow maxdow commented Aug 24, 2015

Provide ability to publish the package to npm .

I make a release here : https://www.npmjs.com/package/ns-popover

fell free to take the ownership

close #92

@nohros
Copy link
Owner

nohros commented Aug 24, 2015

Why the package name needs to be changed?

I am not familiar with nodejs and do not know if this library could be used in node directly, can you clarify the use case?

@maxdow
Copy link
Author

maxdow commented Aug 24, 2015

npm is a package manager which could be used for server and front end.

You can build a frontend app throught browserify or webpack and use a node module with require() call like you do in a nodejs app.

Even if the package is not npm compliant ( it doesn't export a module ) you could use npm to share and download it like you do with bower.

In my case i have ns-popover in dependencies in my package.json. When i type npm install, all my dependencies are loaded from npm.
Now if i use browserify or webpack i just have to require ns-popover in my code and it's loaded exactly as if i added a <script> tag

The name need to be changed because it is in package.json specification :

New packages must not have uppercase letters in the name

https://docs.npmjs.com/files/package.json

I hope this help you to understand ?

@nohros
Copy link
Owner

nohros commented Aug 24, 2015

Ok, thanks for the explanation. I really need to rename the package? This will cause conflicts with bower users that already use the nsPopover.

@maxdow
Copy link
Author

maxdow commented Sep 17, 2015

On npm yes , you can't use camelCase :

"New packages must not have uppercase letters in the name."

https://docs.npmjs.com/files/package.json

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 this pull request may close these issues.

Make this package availabe on npm
2 participants