Skip to content
This repository has been archived by the owner on Aug 4, 2021. It is now read-only.

Commit

Permalink
Rename dynamicRequires -> dynamicRequireTargets
Browse files Browse the repository at this point in the history
  • Loading branch information
lukastaegert committed Oct 1, 2018
1 parent 2749e2e commit c8269ec
Show file tree
Hide file tree
Showing 12 changed files with 13 additions and 13 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,11 +64,11 @@ export default {

// some modules have dynamic `require` calls in them,
// or even circular dependencies (which are not handled well by EJS)
// so including them as `dynamicRequires` will simulate a CJS environment
// so including them as `dynamicRequireTargets` will simulate a CJS environment
// for them with support for dynamic and circular dependencies.
// note that this might hardcode some paths from your PC in the final js file,
// so you might want to search & replace the base path of the root js file.
dynamicRequires: [ // accepts either a string, or an array of strings
dynamicRequireTargets: [ // accepts either a string, or an array of strings
// include using a glob pattern (either a string or an array of strings)
'node_modules/logform/*.js',

Expand Down
2 changes: 1 addition & 1 deletion src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export default function commonjs(options = {}) {
const ignoreGlobal = options.ignoreGlobal;

const { dynamicRequireModuleSet, dynamicRequireModuleDirPaths } = getDynamicRequirePaths(
options.dynamicRequires
options.dynamicRequireTargets
);

const customNamedExports = {};
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require-absolute-import/_config.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
module.exports = {
description: 'resolves non-relative paths via node_modules',
pluginOptions: {
dynamicRequires: [
dynamicRequireTargets: [
'function/dynamic-require-absolute-import/sub/node_modules/module/direct.js',
'function/dynamic-require-absolute-import/sub/node_modules/module/nested/nested.js',
'function/dynamic-require-absolute-import/node_modules/parent-module/parent.js'
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require-es-entry/_config.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module.exports = {
description: 'works when the entry point is an es module',
pluginOptions: {
dynamicRequires: ['function/dynamic-require-es-entry/submodule.js']
dynamicRequireTargets: ['function/dynamic-require-es-entry/submodule.js']
}
};
2 changes: 1 addition & 1 deletion test/function/dynamic-require-extensions/_config.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module.exports = {
description: 'returns the same module instance if requiring with and without extension',
pluginOptions: {
dynamicRequires: ['function/dynamic-require-extensions/submodule.js']
dynamicRequireTargets: ['function/dynamic-require-extensions/submodule.js']
}
};
2 changes: 1 addition & 1 deletion test/function/dynamic-require-globs/_config.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
module.exports = {
description: 'supports glob patterns',
pluginOptions: {
dynamicRequires: [
dynamicRequireTargets: [
'function/dynamic-require-globs/s*.js',
'function/dynamic-require-globs/e*.*',
'!function/dynamic-require-globs/e*2.js'
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require-instances/_config.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ module.exports = {
plugins: [nodeResolve()]
},
pluginOptions: {
dynamicRequires: [
dynamicRequireTargets: [
'function/dynamic-require-instances/direct',
'function/dynamic-require-instances/direct/index.js',
'function/dynamic-require-instances/package',
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require-json/_config.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@ module.exports = {
plugins: [json()]
},
pluginOptions: {
dynamicRequires: ['function/dynamic-require-json/dynamic.json']
dynamicRequireTargets: ['function/dynamic-require-json/dynamic.json']
}
};
2 changes: 1 addition & 1 deletion test/function/dynamic-require-package/_config.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ module.exports = {
plugins: [nodeResolve()]
},
pluginOptions: {
dynamicRequires: [
dynamicRequireTargets: [
'function/dynamic-require-package',
'function/dynamic-require-package/sub',
'function/dynamic-require-package/node_modules/custom-module'
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require-relative-paths/_config.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
module.exports = {
description: 'resolves both windows and posix paths',
pluginOptions: {
dynamicRequires: [
dynamicRequireTargets: [
'function/dynamic-require-relative-paths/sub/submodule.js',
'function/dynamic-require-relative-paths/sub/subsub/subsubmodule.js'
]
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require-resolve-index/_config.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ module.exports = {
plugins: [nodeResolve()]
},
pluginOptions: {
dynamicRequires: [
dynamicRequireTargets: [
'function/dynamic-require-resolve-index',
'function/dynamic-require-resolve-index/sub',
'function/dynamic-require-resolve-index/node_modules/custom-module'
Expand Down
2 changes: 1 addition & 1 deletion test/function/dynamic-require/_config.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module.exports = {
description: 'supports dynamic require',
pluginOptions: {
dynamicRequires: ['function/dynamic-require/submodule.js']
dynamicRequireTargets: ['function/dynamic-require/submodule.js']
}
};

0 comments on commit c8269ec

Please sign in to comment.