Update workflows and resolve lint errors

Planeshifter · Planeshifter · commit 3894b362c902 · 2022-10-10T16:05:28.000-04:00
diff --git a/.github/workflows/lint_changed_files.yml b/.github/workflows/lint_changed_files.yml
@@ -188,6 +188,13 @@ jobs:
             make lint-python-files FILES="${files}"
           fi
 
+      # Setup R:
+      - name: 'Setup R'
+        if: success() || failure()
+        uses: r-lib/actions/setup-r@v2
+        with:
+          r-version: '3.5.3'
+
       # Lint R files:
       - name: 'Lint R files'
         if: success() || failure()
diff --git a/.github/workflows/lint_random_files.yml b/.github/workflows/lint_random_files.yml
@@ -283,9 +283,10 @@ jobs:
 
       # Setup R:
       - name: 'Setup R'
+        if: ( github.event.inputs.r != 'false' ) && ( success() || failure() )
         uses: r-lib/actions/setup-r@v2
         with:
-          r-version: '3.5.3' # The R version to download (if necessary) and use.
+          r-version: '3.5.3'
 
       # Lint R files:
       - name: 'Lint R files'
diff --git a/lib/node_modules/@stdlib/_tools/github/rank-users/lib/validate.js b/lib/node_modules/@stdlib/_tools/github/rank-users/lib/validate.js
@@ -52,13 +52,13 @@ function validate( opts, options ) {
 	if ( hasOwnProp( options, 'token' ) ) {
 		opts.token = options.token;
 		if ( !isString( opts.token ) ) {
-			return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'token',  opts.token ) );
+			return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'token', opts.token ) );
 		}
 	}
 	if ( hasOwnProp( options, 'useragent' ) ) {
 		opts.useragent = options.useragent;
 		if ( !isString( opts.useragent ) ) {
-			return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'useragent',  opts.useragent ) );
+			return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'useragent', opts.useragent ) );
 		}
 	}
 	if ( hasOwnProp( options, 'method' ) ) {
diff --git a/lib/node_modules/@stdlib/_tools/licenses/header/lib/main.js b/lib/node_modules/@stdlib/_tools/licenses/header/lib/main.js
@@ -51,7 +51,7 @@ var header = require( './header.js' );
 *     'year': 2018,
 *     'copyright': 'The Stdlib Authors'
 * };
-* var header = licenseHeader( 'Apache-2.0', 'js', opts );
+* var out = licenseHeader( 'Apache-2.0', 'js', opts );
 * // returns <string>
 */
 function licenseHeader( license, type, options ) {
diff --git a/lib/node_modules/@stdlib/_tools/scripts/create_namespace_types.js b/lib/node_modules/@stdlib/_tools/scripts/create_namespace_types.js
@@ -33,7 +33,6 @@ var RE_EOL = require( '@stdlib/regexp/eol' ).REGEXP;
 var removePunctuation = require( '@stdlib/string/remove-punctuation' );
 var readJSON = require( '@stdlib/fs/read-json' ).sync;
 var replace = require( '@stdlib/string/replace' );
-var camelcase = require( '@stdlib/string/camelcase' );
 var currentYear = require( '@stdlib/time/current-year' );
 var rootDir = require( '@stdlib/_tools/utils/root-dir' );
 
@@ -205,7 +204,7 @@ function create( fullPath ) {
 		// Use existing copyright year from definition file if available...
 		defFile = fs.readFileSync( defFilePath, 'utf8' );
 		match = defFile.match( RE_COPYRIGHT_YEAR );
-		copyrightYear = match ? match[ 1 ] : currentYear();
+		copyrightYear = ( match ) ? match[ 1 ] : currentYear();
 	} else {
 		// Use current year as copyright year:
 		copyrightYear = currentYear();
diff --git a/lib/node_modules/@stdlib/assert/instance-of/benchmark/benchmark.js b/lib/node_modules/@stdlib/assert/instance-of/benchmark/benchmark.js
@@ -22,6 +22,7 @@
 
 var bench = require( '@stdlib/bench' );
 var isBoolean = require( '@stdlib/assert/is-boolean' ).isPrimitive;
+var Object = require( '@stdlib/object/ctor' );
 var pkg = require( './../package.json' ).name;
 var instanceOf = require( './../lib' );
 
@@ -35,7 +36,7 @@ bench( pkg, function benchmark( b ) {
 
 	vals = [
 		new Date(),
-		new RegExp( '.*' ),
+		new RegExp( '.*' ), // eslint-disable-line prefer-regex-literals
 		new Array( 10 ),
 		{},
 		'5',
@@ -65,7 +66,7 @@ bench( pkg+'::built-in', function benchmark( b ) {
 
 	vals = [
 		new Date(),
-		new RegExp( '.*' ),
+		new RegExp( '.*' ), // eslint-disable-line prefer-regex-literals
 		new Array( 10 ),
 		{},
 		'5',
diff --git a/lib/node_modules/@stdlib/assert/is-date-object/test/test.js b/lib/node_modules/@stdlib/assert/is-date-object/test/test.js
@@ -123,7 +123,7 @@ tape( 'the function returns `false` if not provided a `Date` object', function t
 		[],
 		{},
 		/./,
-		new RegExp( '.' ),
+		new RegExp( '.' ), // eslint-disable-line prefer-regex-literals
 		function noop() {},
 		function Date() {}
 	];
diff --git a/lib/node_modules/@stdlib/assert/is-falsy/benchmark/benchmark.js b/lib/node_modules/@stdlib/assert/is-falsy/benchmark/benchmark.js
@@ -47,7 +47,7 @@ bench( pkg, function benchmark( b ) {
 		void 0,
 		[],
 		{},
-		new RegExp( '.*' ),
+		new RegExp( '.*' ), // eslint-disable-line prefer-regex-literals
 		new Date(),
 		function noop() {}
 	];
diff --git a/lib/node_modules/@stdlib/assert/is-generator-object-like/test/es2015-generator/index.js b/lib/node_modules/@stdlib/assert/is-generator-object-like/test/es2015-generator/index.js
@@ -18,7 +18,7 @@
 
 'use strict';
 
-/* eslint-disable no-restricted-syntax */
+/* eslint-disable no-restricted-syntax, node/no-unsupported-features/es-syntax */
 
 // MODULES //
 
diff --git a/lib/node_modules/@stdlib/assert/is-generator-object/test/es2015-generator/index.js b/lib/node_modules/@stdlib/assert/is-generator-object/test/es2015-generator/index.js
@@ -18,7 +18,7 @@
 
 'use strict';
 
-/* eslint-disable no-restricted-syntax */
+/* eslint-disable no-restricted-syntax, node/no-unsupported-features/es-syntax */
 
 // MODULES //
 
diff --git a/lib/node_modules/@stdlib/assert/is-node/lib/main.js b/lib/node_modules/@stdlib/assert/is-node/lib/main.js
@@ -84,6 +84,7 @@ function isNode() {
 		// Check for a `node` property:
 		isString( proc.versions.node ) &&
 
+		/* eslint-disable node/no-unsupported-features/es-builtins */
 		(
 			// `process.release` was added in Node v3.0.0 via io.js:
 			typeof proc.release === 'undefined' ||
@@ -99,6 +100,7 @@ function isNode() {
 				RE.test( proc.release.name )
 			)
 		)
+		/* eslint-enable node/no-unsupported-features/es-builtins */
 	);
 }
 
diff --git a/lib/node_modules/@stdlib/assert/is-nonnegative-integer/test/test.object.js b/lib/node_modules/@stdlib/assert/is-nonnegative-integer/test/test.object.js
@@ -56,7 +56,7 @@ tape( 'the function returns `false` if not provided a number', function test( t
 		{},
 		new Date(),
 		/./,
-		new RegExp( '.' ),
+		new RegExp( '.' ), // eslint-disable-line prefer-regex-literals
 		function noop() {}
 	];
 
diff --git a/lib/node_modules/@stdlib/regexp/native-function/test/test.regexp.js b/lib/node_modules/@stdlib/regexp/native-function/test/test.regexp.js
@@ -25,6 +25,8 @@
 var tape = require( 'tape' );
 var Int8Array = require( '@stdlib/array/int8' );
 var Number = require( '@stdlib/number/ctor' );
+var Boolean = require( '@stdlib/boolean/ctor' );
+var Fcn = require( '@stdlib/function/ctor' );
 var RE_NATIVE_FUNCTION = require( './../lib/regexp.js' );
 
 
@@ -46,7 +48,7 @@ tape( 'the regular expression matches native functions', function test( t ) {
 		Boolean,
 		String,
 		Number,
-		Function,
+		Fcn,
 		RegExp,
 		Number,
 		Int8Array,
diff --git a/lib/node_modules/@stdlib/string/truncate-middle/benchmark/benchmark.js b/lib/node_modules/@stdlib/string/truncate-middle/benchmark/benchmark.js
@@ -57,7 +57,7 @@ function createBenchmark( len ) {
 			}
 		}
 		b.toc();
-		if ( typeof out !== 'string' ) {
+		if ( !isString( out ) ) {
 			b.fail( 'should return a string' );
 		}
 		b.pass( 'benchmark finished' );
diff --git a/lib/node_modules/@stdlib/utils/function-name/examples/index.js b/lib/node_modules/@stdlib/utils/function-name/examples/index.js
@@ -22,6 +22,7 @@
 
 var Float64Array = require( '@stdlib/array/float64' );
 var Buffer = require( '@stdlib/buffer/ctor' );
+var Boolean = require( '@stdlib/boolean/ctor' );
 var Number = require( '@stdlib/number/ctor' );
 var functionName = require( './../lib' );
 
diff --git a/lib/node_modules/@stdlib/utils/property-descriptors/lib/builtin.js b/lib/node_modules/@stdlib/utils/property-descriptors/lib/builtin.js
@@ -18,9 +18,14 @@
 
 'use strict';
 
+// MODULES //
+
+var Object = require( '@stdlib/object/ctor' );
+
+
 // VARIABLES //
 
-var propertyDescriptors = Object.getOwnPropertyDescriptors;
+var propertyDescriptors = Object.getOwnPropertyDescriptors; // eslint-disable-line node/no-unsupported-features/es-builtins
 
 
 // MAIN //
diff --git a/lib/node_modules/@stdlib/utils/property-names/lib/builtin.js b/lib/node_modules/@stdlib/utils/property-names/lib/builtin.js
@@ -18,6 +18,11 @@
 
 'use strict';
 
+// MODULES //
+
+var Object = require( '@stdlib/object/ctor' );
+
+
 // VARIABLES //
 
 var propertyNames = Object.getOwnPropertyNames;
diff --git a/lib/node_modules/@stdlib/utils/property-names/lib/polyfill.js b/lib/node_modules/@stdlib/utils/property-names/lib/polyfill.js
@@ -20,6 +20,7 @@
 
 // MODULES //
 
+var Object = require( '@stdlib/object/ctor' );
 var keys = require( '@stdlib/utils/keys' );
 
 
diff --git a/lib/node_modules/@stdlib/utils/writable-property-names/examples/index.js b/lib/node_modules/@stdlib/utils/writable-property-names/examples/index.js
@@ -44,6 +44,5 @@ defineProperty( Foo.prototype, 'bip', {
 
 var obj = new Foo();
 var keys = writablePropertyNames( obj );
-
 console.log( keys );
 // => [ 'a' ]
diff --git a/lib/node_modules/@stdlib/utils/writable-property-symbols/examples/index.js b/lib/node_modules/@stdlib/utils/writable-property-symbols/examples/index.js
@@ -38,4 +38,4 @@ if ( hasSymbols ) {
 var obj = new Foo();
 var symbols = writablePropertySymbols( obj );
 console.log( symbols );
-// e.g., returns [ Symbol( 'baz' ) ]
+// e.g., => [ Symbol( 'baz' ) ]

Original file line number	Diff line number	Diff line change
`@@ -52,13 +52,13 @@ function validate( opts, options ) {`
`52`	`52`	`if ( hasOwnProp( options, 'token' ) ) {`
`53`	`53`	`opts.token = options.token;`
`54`	`54`	`if ( !isString( opts.token ) ) {`
`55`		- return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'token', opts.token ) );
	`55`	+ return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'token', opts.token ) );
`56`	`56`	`}`
`57`	`57`	`}`
`58`	`58`	`if ( hasOwnProp( options, 'useragent' ) ) {`
`59`	`59`	`opts.useragent = options.useragent;`
`60`	`60`	`if ( !isString( opts.useragent ) ) {`
`61`		- return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'useragent', opts.useragent ) );
	`61`	+ return new TypeError( format( 'invalid option. `%s` option must be a string. Option: `%s`.', 'useragent', opts.useragent ) );
`62`	`62`	`}`
`63`	`63`	`}`
`64`	`64`	`if ( hasOwnProp( options, 'method' ) ) {`