Skip to content

Commit

Permalink
Build: upgrade dependencies; fix bundler tests on windows
Browse files Browse the repository at this point in the history
- account for newly unused parameters in the slim builds

Closes gh-5509
  • Loading branch information
timmywil authored Jun 15, 2024
1 parent ea31e4d commit cb8ab6c
Show file tree
Hide file tree
Showing 5 changed files with 704 additions and 456 deletions.
2 changes: 1 addition & 1 deletion build/tasks/promises_aplus_tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ async function runTests() {
spawn(
command,
[ test ].concat( args ),
{ stdio: "inherit" }
{ shell: true, stdio: "inherit" }
);
} );
}
Expand Down
26 changes: 24 additions & 2 deletions eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@ export default [
ignores: [
"external",
"**/tmp",
"test/data/json_obj.js"
"test/data/json_obj.js",
"test/data/jquery-*.js"
]
},

Expand Down Expand Up @@ -147,7 +148,6 @@ export default [
"test/unit/**"
],
ignores: [
"test/data/jquery-3.7.1.js",
"test/data/badcall.js",
"test/data/badjson.js",
"test/data/support/csp.js",
Expand Down Expand Up @@ -349,6 +349,28 @@ export default [
}
},

{
files: [
"dist/jquery.slim.js",
"dist/jquery.factory.slim.js",
"dist-module/jquery.slim.module.js",
"dist-module/jquery.factory.slim.module.js"
],
rules: {

// Rollup is now smart enough to remove the use
// of parameters if the argument is not passed
// anywhere in the build.
// The removal of effects in the slim build
// results in some parameters not being used,
// which can be safely ignored.
"no-unused-vars": [
"error",
{ args: "none" }
]
}
},

{
files: [
"src/wrapper.js",
Expand Down
Loading

0 comments on commit cb8ab6c

Please sign in to comment.