Skip to content

Commit a6d536c

Browse files
author
emilien.escalle
committed
Improve code of getExternalModules function
1 parent 29f4fc4 commit a6d536c

File tree

1 file changed

+16
-18
lines changed

1 file changed

+16
-18
lines changed

lib/packExternalModules.js

Lines changed: 16 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -153,28 +153,26 @@ function findExternalOrigin(issuer) {
153153
}
154154

155155
function getExternalModules(stats) {
156-
if (stats.compilation.chunks) {
157-
const externals = new Set();
158-
for (const chunk of stats.compilation.chunks) {
159-
if (!chunk.modulesIterable) {
160-
continue;
161-
}
156+
if (!stats.compilation.chunks) {
157+
return [];
158+
}
159+
const externals = new Set();
160+
for (const chunk of stats.compilation.chunks) {
161+
if (!chunk.modulesIterable) {
162+
continue;
163+
}
162164

163-
// Explore each module within the chunk (built inputs):
164-
for (const module of chunk.modulesIterable) {
165-
if (isExternalModule(module)) {
166-
externals.add({
167-
origin: _.get(findExternalOrigin(module.issuer), 'rawRequest'),
168-
external: getExternalModuleName(module)
169-
});
170-
}
165+
// Explore each module within the chunk (built inputs):
166+
for (const module of chunk.modulesIterable) {
167+
if (isExternalModule(module)) {
168+
externals.add({
169+
origin: _.get(findExternalOrigin(module.issuer), 'rawRequest'),
170+
external: getExternalModuleName(module)
171+
});
171172
}
172173
}
173-
174-
return Array.from(externals);
175174
}
176-
177-
return [];
175+
return Array.from(externals);
178176
}
179177

180178
module.exports = {

0 commit comments

Comments
 (0)