From 82f2b76e25a4a67e01ec67e5ebe39d14ad771ea8 Mon Sep 17 00:00:00 2001 From: Florian Dold Date: Thu, 20 Apr 2017 03:09:25 +0200 Subject: Reorganize module loading. We now use webpack instead of SystemJS, effectively bundling modules into one file (plus commons chunks) for every entry point. This results in a much smaller extension size (almost half). Furthermore we use yarn/npm even for extension run-time dependencies. This relieves us from manually vendoring and building dependencies. It's also easier to understand for new developers familiar with node. --- .../systemjs/test/tests/commonjs-requires.js | 40 ---------------------- 1 file changed, 40 deletions(-) delete mode 100644 thirdparty/systemjs/test/tests/commonjs-requires.js (limited to 'thirdparty/systemjs/test/tests/commonjs-requires.js') diff --git a/thirdparty/systemjs/test/tests/commonjs-requires.js b/thirdparty/systemjs/test/tests/commonjs-requires.js deleted file mode 100644 index af80d8058..000000000 --- a/thirdparty/systemjs/test/tests/commonjs-requires.js +++ /dev/null @@ -1,40 +0,0 @@ -exports.d6 = -/**/require('./commonjs-d2.js'); - -exports.d1 = require( - './commonjs-d.js' -); - -exports.d2 = (require -("./commonjs-d.js")); - -var regex = / \/* /; - -exports.d3 = "require('not a dep')"; - -exports.d4 = "text/* require('still not a dep') text"; - -exports.d5 = 'text \'quote\' require("yet still not a dep")'; - -var regexWithString = /asdfasdf " /; - -var regexClose = /asdf " */; - -// This comment triggered SystemJS to do a require because of this -> require('') -exports.d7 = 'export'; - -var p = false && require('" + "test" + "'); - -// this line shouldn't be detected -" = require(", "),\n "; - - -/* - -Unsolved breaking cases: - -var regex = / " /; var string = " /* " // one line; -require('asdf') // <- this will now be skipped as it will be in the '/*' comment -*//* - -*/ \ No newline at end of file -- cgit v1.2.3