-
Notifications
You must be signed in to change notification settings - Fork 304
Add "standalone" option to component.json #512
Comments
This has come up a few times in the past and I think the idea has been turned down. The reason being that the builder shouldn't store build options within the manifest because it isn't relevant information about the package. It could create weird cases when building with packages that have dependencies with options like these. |
oh i don't think you need this property when building your own component. |
I get the idea that the component.json file, as it is related to the component and not the build process, probably shouldn't have config info for the component-builder, but I think the need for a that type of config exists and should be considered. I imagine that the absence of this configuration is the reason why we see people putting Makefiles in their repos a lot. I think there should be a more convenient way to manage building considering the frequency with which it is done during development. Additionally, I think it's a steep enough learning curve to get on board with the philosophy of component let alone ask devs to use Make - I think 1 out of 4 or 5 front-end devs (maybe grim outlook) actually know/understand/have seen Makfiles, it will probably intimidate some |
I'd be open to trying it out. But I still think it messes with concerns a bit too much. It could be nice for "boot" components. Maybe open a pull request instead. @visionmedia and @ianstormtaylor are going to hate this though :) {
"builder": {
"standalone": "myapp"
...
}
} |
we could also use a |
i'm going to add |
Based on convo at #511 (comment)
When doing
or
component builder should look for a property called "standalone": true, which would make it so you don't have to even specify the option, and look to the name of the component as the name of the exported module.
Additionally, we may want to be able to override the name of the standalone build with a property like standaloneName, outputName, umdName, something along those lines. I would picture it a couple of ways:
The text was updated successfully, but these errors were encountered: