Add to bower

[callumacrae]

bower install dragdealer currently links to this: https://github.com/yahyaKacem/dragdealer, which is out of date.

[ovidiuch]

Well we could create the bower config in this repo but I think @yahyaKacem would have to unregister their repo to publish it again with the official repo. I'm not very familiar with Bower but I found this bower/bower#120

[callumacrae]

If @yahyaKacem doesn't unregister their repo, you could always publish it as something like "dragdealer.js". Hopefully they will, though.

[yahyaKacem]

sorry I don't know how to unregister it, if you can point me to a link describing how, i'll do it.

[callumacrae]

Post on bower/bower#120 asking for your dragdealer package to be removed. Thanks!

[ooxi]

How cool is that, I just wanted to ask you to get in contact with @yahyaKacem but he already responeded!

[ovidiuch]

@ooxi yup, you can tell the real value of GitHub in moments like this. Now it's just a matter of time until the guys at @bower do their manual labor :)

[ovidiuch]

I can see it takes a while for the bower people to unregister a repo. In the meantime, @callumacrae or anyone else, feel free to create a small pull request with a minimal bower.json config in place (should be very similar with the npm one)

[ooxi]

@yahyaKacem since bower seems to take ages to update their repository, could you clone skidding's?

[ovidiuch]

Not a bad idea, do you think it would work if @yahyaKacem removes their
repo and forks dragdealer instead?

On Sunday, March 16, 2014, ooxi notifications@github.com wrote:

@yahyaKacem https://github.com/yahyaKacem since bower seems to take
ages to update their repository, could you clone skidding's?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/27#issuecomment-37751049
.

Best regards,
Ovidiu Cherecheş

[yahyaKacem]

Check it now.

[simshanith]

So, first off, thanks for publishing such an awesome update, and making it available here. I just pulled in the latest with Bower, and now it doesn't include the minified lib/dragdealer.min.js thanks to the lib/ ignore rule. Not sure how you want to handle this; but I think it's nice to have available to bower users.

[ovidiuch]

The main path points to src/dragdealer.js, why is that no good? I'm not sure what the best Bower flow is but I would expect to minify all files inside a project myself using grunt/gulp/etc. - I looked at other libraries and saw that they point to the unminified source as well in their bower config. Let me know what you think

[yahyaKacem]

Here I added the minified file.

[simshanith]

Thanks @yahyaKacem !

@skidding , main as src/dragdealer.js is good. I'm not quite sure what the best Bower flow is either, so erring on the side of flexibility makes sense to me. On some projects, simply concatenating vendor-provided minified scripts is all you need. Some vendor projects may minify with Google Closure Compiler or have special Uglify flags (etc.) that I, as a vendor library consumer, don't want to worry about. By including a minified file that passes unit tests the library consumer doesn't need to worry about the minification implementation, which I think is a good thing.