The primary concern is addressed in dac833d, which tags the node-html module as copyOnly instead of marking it as not AMD. Given that this is a module intended for use with node, it's kind of superfluous for the build to process it anyway. This resolves two things:
The build will no longer improperly wrap this module with AMD boilerplate with dojo, dijit, and dojox
The build will no longer complain about this module if dijit or dojox are not included as packages (since the module will no longer be processed and improperly wrapped)
The other commit moves the regular expression for the amd resource tag into a variable, matching how the test resource tag works.
kriszyp
commented
11 years ago
This pull request includes 2 commits.
The primary concern is addressed in dac833d, which tags the
node-htmlmodule ascopyOnlyinstead of marking it as not AMD. Given that this is a module intended for use with node, it's kind of superfluous for the build to process it anyway. This resolves two things:dijitordojoxare not included as packages (since the module will no longer be processed and improperly wrapped)The other commit moves the regular expression for the
amdresource tag into a variable, matching how thetestresource tag works.