Skip to content

Commit

Permalink
Improve "checkDependency"
Browse files Browse the repository at this point in the history
  • Loading branch information
JiuqingSong committed Sep 20, 2023
1 parent 5cef7ba commit d8fbc78
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions tools/buildTools/checkDependency.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ function getPossibleNames(dir, objectName) {
];
}

function processFile(dir, filename, files, externalDependencies) {
function processFile(dir, filename, files, externalDependencies, fromFile) {
if (
externalDependencies.some(d => (typeof d === 'string' ? d == filename : d.test(filename)))
) {
Expand All @@ -27,7 +27,8 @@ function processFile(dir, filename, files, externalDependencies) {
filename +
' under ' +
dir +
': File not found'
': File not found. Source file: ' +
fromFile
);
}

Expand All @@ -54,7 +55,7 @@ function processFile(dir, filename, files, externalDependencies) {
while ((match = reg.exec(content))) {
var nextFile = match[1];
if (nextFile) {
processFile(dir, nextFile, files, externalDependencies);
processFile(dir, nextFile, files, externalDependencies, thisFilename);
}
}

Expand Down Expand Up @@ -95,11 +96,13 @@ function checkDependency() {
var peerDependencies = packageJson.peerDependencies
? Object.keys(packageJson.peerDependencies)
: [];
const startFile = path.join(packageName, 'lib/index');
processFile(
packageRoot,
path.join(packageName, 'lib/index'),
startFile,
[],
dependencies.concat(peerDependencies).concat(GlobalAllowedCrossPackageDependency)
dependencies.concat(peerDependencies).concat(GlobalAllowedCrossPackageDependency),
startFile + '.ts'
);
});
}
Expand Down

0 comments on commit d8fbc78

Please sign in to comment.