Exclude external resources from r.js optimization#208
Exclude external resources from r.js optimization#208dchekanov wants to merge 2 commits intoguybedford:masterfrom
Conversation
|
This fix works for me please prioritize a merge |
|
@dchekanov I reviewed this PR a bit, and I will do more deep research how it might affect end customers. Meanwhile, if u still available for this module, could u please think on test case to write in test.js (or let me know what exactly test result u relied on). If no reply during the week, I will recreate PR from my own with suggested code changes. |
|
@alundiak this plugin currently doesn't have any tests for the building part, so I'm not sure how the suggested change should be tested. There are 2 things that this PR does to support loading modules via the
|
First 2 will show us at least, if all files (css.js, css-builder.js, normalize.js) working file. And I will think about this steps later and provide pr with test for such test cases in test.js. |
Fixes #207.