Invalid JS identifier in exports parameter use case

[andvgal]

The "exports" option may be set to invalid identifier (e.g. with dash). Invalid JS code would get generated in UMD:

this.exports-value = factory(dep1, dep2);

I am not completely sure how valid is such use case, but the generated code can be changed to:

this["exports-value"] = factory(dep1, dep2);

[RReverser]

I'm not sure if it's critical - why would anyone wrap UMD with a name that can be later accessed also only as window['j-query'] and not just jQuery? But PR is welcome anyway.

[andvgal]

I agree, it is not a priority at all.
Obviously, it does not make much sense for jQuery, but it may be used on intention not to pullute the global namespace (e.g. for plugin which needs to be referenced directly only once).
Can you please assign the ticket to me, so I don't forget about PR?

[RReverser]

Unfortunately tickets can be assigned only to project owners, so I'd recommend to track it separately.