Read component's `"main"` from `package.json`, if `bower.json` doesn't have it about bower-requirejs HOT 14 CLOSED

Is there some reason package.json can't be used as a fall back ?

This has been brought up many times in the Bower issue tracker and has been declined. This is not the place to implement it.

Also it would be usefull to display which components failed to be associated with a .js

Sure, that could be useful. @robdodson what do you think?

from bower-requirejs.

Yeah it would be easy to list out the components that didn't make the cut. I'll see if I can put together a PR sometime this week.

from bower-requirejs.

I don't see why this isn't the place to implement grabbing main from a package.json file. You're already assuming a top-level js file is the main file; how is that any better?

from bower-requirejs.

@mlmorg because package.json is for node and its main might not be the same as the browser one or even work.

from bower-requirejs.

Sure, but doesn't a top-level js file run the same risk?

from bower-requirejs.

@mlmorg you're right.

from bower-requirejs.

I'm not opposed to adding it. Although I think the task itself could probably use some house cleaning. It's starting to turn into two giant callbacks.

from bower-requirejs.

@robdodson definitely

from bower-requirejs.

Does anyone have an example of a library that defines a main js file in package.json but not bower.json? Just so I can put together a test fixture. @anodynos @mlmorg ?

from bower-requirejs.

Backbone.js?

from bower-requirejs.

@mlmorg perfect, thanks!

from bower-requirejs.

also Underscore.js has no bower.json, but has package.json and main in it, though both of them work fine at the moment, since they have main js-file names same way as package, but for example backbone-forms doesn't have bower.json and has no main js-file named as package, so it's simply ignored, though it has main in package.json, which can be used by bower-requirejs to provide path for Require.js

from bower-requirejs.

This should be addressed by this commit in the postinstall-refactor branch (robdodson/grunt-bower-requirejs@1779f4d). There's a test for backbone-amd which doesn't have a bower.json but does have a package.json which properly specifies the location of the main file. Feel free to PR additional tests if you'd like.

from bower-requirejs.

Fixed in v0.8.0

from bower-requirejs.