Skip to content
This repository was archived by the owner on Oct 4, 2022. It is now read-only.

Commit 2c4191d

Browse files
committed
Address CR and lint warnings
1 parent 9ae062c commit 2c4191d

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

scripts/list-dependencies.js

+15-14
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
#!/usr/local/bin/node
2+
/* eslint-disable no-console */
23
const writeFileSync = require( "fs" ).writeFileSync;
34
const execSync = require( "child_process" ).execSync;
45

@@ -59,7 +60,7 @@ function whyPackage( name, ignoreApps = true ) {
5960
*
6061
* @param {string[]} packages The list of packages.
6162
*
62-
* @returns {{name: string, dependencies: string[]}[]} The list of dependencies.
63+
* @returns {{name: string, dependencyOf: string[]}[]} The list of dependencies.
6364
*/
6465
function createDependencyList( packages ) {
6566
const padAmount = packages.length % 10;
@@ -70,7 +71,7 @@ function createDependencyList( packages ) {
7071
console.info( `${ ( index + 1 ).toString().padStart( padAmount, "0" ) }/${ packages.length }: ${ name }` );
7172
return {
7273
name,
73-
dependencies: whyPackage( name ),
74+
dependencyOf: whyPackage( name ),
7475
};
7576
} );
7677
console.groupEnd();
@@ -88,8 +89,8 @@ function createDependencyList( packages ) {
8889
function convertToMarkDown( data ) {
8990
return "| This package: | Is a dependency of: |\n|---|---|\n" +
9091
data
91-
.map( ( { name, dependencies } ) => {
92-
const deps = dependencies.length === 0 ? "-" : dependencies.join( "<br>" );
92+
.map( ( { name, dependencyOf } ) => {
93+
const deps = dependencyOf.length === 0 ? "-" : dependencyOf.join( "<br>" );
9394
return `| ${ name } | ${ deps } |`;
9495
} )
9596
.join( "\n" );
@@ -100,6 +101,8 @@ function convertToMarkDown( data ) {
100101
*
101102
* @param {string} filename The filename of the file.
102103
* @param {string} data The data to write.
104+
*
105+
* @returns {void}
103106
*/
104107
function writeFile( filename, data ) {
105108
try {
@@ -111,19 +114,17 @@ function writeFile( filename, data ) {
111114
}
112115

113116
const params = process.argv.splice( 2 );
114-
const packages = getPackages();
115-
let filename = "dependencies.";
116-
let data = createDependencyList( packages );
117+
let outputFilename = "dependencies.";
118+
let outputData = createDependencyList( getPackages() );
117119

118120
if ( params.includes( "--markdown" ) ) {
119-
filename += "md";
121+
outputFilename += "md";
120122
console.info( "Converting to markdown data" );
121-
data = convertToMarkDown( data );
122-
}
123-
else {
124-
filename += "json";
123+
outputData = convertToMarkDown( outputData );
124+
} else {
125+
outputFilename += "json";
125126
console.info( "Converting to plain text" );
126-
data = JSON.stringify( data, null, 2 );
127+
outputData = JSON.stringify( outputData, null, 2 );
127128
}
128129

129-
writeFile( filename, data );
130+
writeFile( outputFilename, outputData );

0 commit comments

Comments
 (0)