Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

NPM support #36

Closed
wants to merge 1 commit into from
Closed

NPM support #36

wants to merge 1 commit into from

Conversation

vp-hwood
Copy link

With this, npm install angular-ui/bootstrap-bower becomes possible. This is of course only temporary until real NPM packages are managed by the Angular UI Bootstrap team.

@msieurtoph
Copy link

Merge it, merge it, merge it ! ;)

@christopherthielen
Copy link

+1 trying to rid myself of bower

@dansomething
Copy link

+1 I suppose this repo would have to be renamed since it would no longer be strictly for Bower, but I'd be fine with that as long as we got official NPM support.

@gsemet
Copy link

gsemet commented Apr 7, 2015

would be better if bower install could work automatically

@goldcaddy77
Copy link

Looks like somebody already has angular-bootstrap, unfortunately. Have you tried registering on npm?

@vp-hwood
Copy link
Author

Seems like @leon has published a package on NPM since I opened this issue.

@leon
Copy link

leon commented Apr 10, 2015

I don't understand why we need a separate repo for bower?
Why cant we just use the normal repo?

The npm repo was not being updated so I took over and published a version.
I also gave the authors right to deploy to it, so it shouldn't be a problem for them to take over.

Is there anyone else that should have publish rights on the package?

@goldcaddy77
Copy link

Are any of you core angular-ui/bootstrap contributors? I'd be more than happy to put in a PR to get things into the main angular-ui/bootstrap project, but I'm not sure if they'd be interested. It would just need a quick build process set up to generate the dist files and moving the links in the 2 registries.

@goldcaddy77
Copy link

Btw, I registered the npm angular-ui-bootstrap project last night so that we'd have it in case we move forward.

@karianna
Copy link

I've let the maintainers know.

@nl0
Copy link

nl0 commented May 21, 2015

hey guys, what's up? it would be really nice to have this merged

],
"license": "MIT",
"ignore": [],
"main": ["./ui-bootstrap-tpls.js"],
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this one breaks browserify -- it should be one file, not an array: "main": "./ui-bootstrap-tpls.js",

@burdiyan
Copy link

Hi there! What is stopping the core team from merge this PR? It would be so nice to have this on npm.

@Foxandxss
Copy link
Contributor

Let's continue this on #60 and lets hope for a merge today.

@Foxandxss Foxandxss closed this Jul 22, 2015
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.

None yet