Use anonymous module when exporting via AMD

[yortus]

Knockstrap currently uses a hardcoded module name when exporting via AMD, which is not recommended practice. I noticed this when trying to use knockstrap, but finding it would only load if served from the root path, because that is the only way the require()'d module ID matches the hardcoded name.

This change only impacts users loading knockstrap via AMD. It does not affect them if they serve knockstrap from the root path (ie at /knockstrap.js), which is the only way it works normally anyway. Anyone else loading knockstrap via AMD on a non-root path would already be using path and map config to work around the problem, so they should also be unaffected. Basically there is little downside to this change.

Here are some further references:

• RequireJS documentation advising against named modules
• similar problem with moment.js leading eventually to them switching to anonymous AMD export
• moment.js PR switching to anonymous module
• chai.js issue: Self Named Module is not good
• RequireJS wiki on why to use anonymous modules
• A StackOverflow Q&A on the topic

Examples of other knockout plugins (all using anonymous module defines):

• knockout.punches
• knockout-projections
• knockout-es5

[faulknercs]

Thanks for contribution!

[yortus]

Great, thanks! Any idea when your latest changes will make it onto bower?

[faulknercs]

I think, I'll make new release in next two weeks