Skip to content

Commit

Permalink
Build: Remove the external directory, read from node_modules directly
Browse files Browse the repository at this point in the history
Now that Sizzle is gone & we use npm, we can read from node_modules directly
and skip the setup that copies some files to the external directory.
  • Loading branch information
mgol committed Aug 21, 2019
1 parent ac5f7cd commit 342c551
Show file tree
Hide file tree
Showing 12 changed files with 10 additions and 41,508 deletions.
4 changes: 0 additions & 4 deletions .npmignore
Expand Up @@ -10,7 +10,3 @@
/speed
/test
/Gruntfile.js

/external/qunit
/external/requirejs
/external/sinon
28 changes: 4 additions & 24 deletions Gruntfile.js
Expand Up @@ -68,25 +68,6 @@ module.exports = function( grunt ) {
}
}
},
npmcopy: {
all: {
options: {
destPrefix: "external"
},
files: {
"npo/npo.js": "native-promise-only/lib/npo.src.js",

"qunit/qunit.js": "qunit/qunit/qunit.js",
"qunit/qunit.css": "qunit/qunit/qunit.css",
"qunit/LICENSE.txt": "qunit/LICENSE.txt",

"requirejs/require.js": "requirejs/require.js",

"sinon/sinon.js": "sinon/pkg/sinon.js",
"sinon/LICENSE.txt": "sinon/LICENSE"
}
}
},
jsonlint: {
pkg: {
src: [ "package.json" ]
Expand Down Expand Up @@ -161,9 +142,9 @@ module.exports = function( grunt ) {
],
files: [
"test/data/jquery-1.9.1.js",
"external/sinon/sinon.js",
"external/npo/npo.js",
"external/requirejs/require.js",
"node_modules/sinon/pkg/sinon.js",
"node_modules/native-promise-only/lib/npo.src.js",
"node_modules/requirejs/require.js",
"test/data/testinit.js",

"test/jquery.js",
Expand Down Expand Up @@ -196,7 +177,7 @@ module.exports = function( grunt ) {

{ pattern: "dist/jquery.*", included: false, served: true },
{ pattern: "src/**", included: false, served: true },
{ pattern: "external/**", included: false, served: true },
{ pattern: "node_modules/**", included: false, served: true },
{
pattern: "test/**/*.@(js|css|jpg|html|xml|svg)",
included: false,
Expand Down Expand Up @@ -231,7 +212,6 @@ module.exports = function( grunt ) {
// jsdom only runs basic tests
"test/unit/basic.js",

{ pattern: "external/**", included: false, served: true },
{
pattern: "test/**/*.@(js|css|jpg|html|xml|svg)",
included: false,
Expand Down

0 comments on commit 342c551

Please sign in to comment.