Require callback signature to have two parameters

Planeshifter · Planeshifter · commit 3ab7917126ec · 2020-12-15T19:56:49.000-05:00
diff --git a/lib/node_modules/@stdlib/fs/resolve-parent-path/docs/types/index.d.ts b/lib/node_modules/@stdlib/fs/resolve-parent-path/docs/types/index.d.ts
@@ -28,33 +28,13 @@ interface Options {
 	dir?: string;
 }
 
-/**
-* Callback invoked after resolving a path.
-*/
-type Nullary = () => void;
-
-/**
-* Callback invoked after resolving a path.
-*
-* @param err - error argument
-*/
-type Unary = ( err: Error ) => void;
-
-/**
-* Callback invoked after resolving a path.
-*
-* @param err - error argument
-* @param path - resolved path
-*/
-type Binary = ( err: Error, path: string | null ) => void;
-
 /**
 * Callback invoked after resolving a path.
 *
 * @param err - error argument
 * @param path - resolved path
 */
-type Callback = Nullary | Unary | Binary;
+type Callback = ( err: Error | null, path: string | null ) => void;
 
 /**
 * Interface for resolving a path by walking parent directories.
diff --git a/lib/node_modules/@stdlib/fs/resolve-parent-path/docs/types/test.ts b/lib/node_modules/@stdlib/fs/resolve-parent-path/docs/types/test.ts
@@ -18,7 +18,7 @@
 
 import resolveParentPath = require( './index' );
 
-const done = ( error: Error, path: string | null ) => {
+const done = ( error: Error | null, path: string | null ) => {
 	if ( error || path === null ) {
 		throw error;
 	}
@@ -30,7 +30,6 @@ const done = ( error: Error, path: string | null ) => {
 // The function does not have a return value...
 {
 	resolveParentPath( 'package.json', done ); // $ExpectType void
-	resolveParentPath.sync( 'package.json' ); // $ExpectType string | null
 }
 
 // The compiler throws an error if the function is provided a first argument which is not a string...

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@`
`18`	`18`
`19`	`19`	`import resolveParentPath = require( './index' );`
`20`	`20`
`21`		`-const done = ( error: Error, path: string \| null ) => {`
	`21`	`+const done = ( error: Error \| null, path: string \| null ) => {`
`22`	`22`	`if ( error \|\| path === null ) {`
`23`	`23`	`throw error;`
`24`	`24`	`}`
`@@ -30,7 +30,6 @@ const done = ( error: Error, path: string \| null ) => {`
`30`	`30`	`// The function does not have a return value...`
`31`	`31`	`{`
`32`	`32`	`resolveParentPath( 'package.json', done ); // $ExpectType void`
`33`		`- resolveParentPath.sync( 'package.json' ); // $ExpectType string \| null`
`34`	`33`	`}`
`35`	`34`
`36`	`35`	`// The compiler throws an error if the function is provided a first argument which is not a string...`