Skip to content

Commit 525d8af

Browse files
authored
Merge pull request #16 from js-kyle/revert-15-master
Revert "Issue #12"
2 parents ac406ba + 8b03132 commit 525d8af

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

index.js

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -9,17 +9,15 @@ const walkModules = ((path = 'node_modules') => {
99
dirTree(path, {extensions:/\.json$/}, (item) => {
1010

1111
if (item.name === 'package.json') {
12-
try{
13-
let pkg = JSON.parse(fs.readFileSync(item.path, 'utf8'));
14-
if (!pkg || !pkg.scripts) return;
15-
let scripts = Object.keys(pkg.scripts);
16-
scripts.forEach((script) => {
17-
if (scriptTypes.includes(script)) {
18-
reports.push({ name: pkg.name, script: script, src:item.path })
19-
}
20-
});
21-
catch(e){
22-
}
12+
let pkg = JSON.parse(fs.readFileSync(item.path, 'utf8'));
13+
if (!pkg.scripts) return;
14+
let scripts = Object.keys(pkg.scripts);
15+
scripts.forEach((script) => {
16+
if (scriptTypes.includes(script)) {
17+
reports.push({name: pkg.name, script: script})
18+
}
19+
});
20+
2321
}
2422

2523
});
@@ -32,7 +30,7 @@ process.on('exit', () => {
3230
} else {
3331
console.log('\x1b[31m', 'Potentially unsafe scripts found. These should be reviewed for safety', '\x1b[0m');
3432
reports.forEach((report) => {
35-
console.log(`Module name: ${report.name} Type: ${report.script} Src: ${report.src}` );
33+
console.log(`Module name: ${report.name} Type: ${report.script}`);
3634
});
3735
}
3836

0 commit comments

Comments
 (0)