Issues installing via Bower and Grunt's wiredep about angular-chartjs HOT 6 CLOSED

it should be:

"dist/angular-chartjs.js"

from angular-chartjs.

dist/ is new since I opened this issue. I'm not sure whether bower should point at that or not, though. I don't often see dist in a repo--that's usually something created by a build tool.

from angular-chartjs.

Hello gentlemen,

I am in the middle of creating a new release for this library that compatible with the new versions of ChartJS. What you see here is a work in progress (although the code is now more or less stable). I need to release it properly and write some basic tests.

@JAAulde
As far as having dist/ in the repo goes, I generally don't include it for repos that I know will only be used as standalone NodeJS packages, especially on the server / application side. It is automatically created by the GulpJS build tasks.

The reason I DO however include it for packages like this one (that are client-side only) is because many people (using Bower or otherwise) like to have quick and easy access to the built files without having to do any extra work for including the package into their dependency structure (in the HTML).

While this can cause merge conflicts from time to time with many collaborators, it is usually fairly trivial to work out.

I will update Bower in line with the coming release. If not today, then over this weekend.

from angular-chartjs.

Hi Peter, thanks for your work and the info. Please don't think I was criticizing the presence of dist/ in the repo, I was just talking through where the main property ought to point and what I'm accustomed to seeing. I have one distributed browser-only library and some others in the works and have considered putting a dist/ in the repo as well.

Cheers,
Jim

from angular-chartjs.

@JAAulde Oh yeah no worries I just wanted to make it clear what the purpose of it was, no criticism suspected :)

from angular-chartjs.

The versions and paths have been updated.

from angular-chartjs.