From 7c20aee25e23d236ada061057ecb24968e9ef6f2 Mon Sep 17 00:00:00 2001 From: Krishna Sharma Date: Thu, 30 Jul 2026 09:14:43 +0000 Subject: [PATCH 1/3] fix: resolve JavaScript lint errors in three packages --- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown_pkg_readmes status: na - task: lint_markdown_docs status: na - task: lint_markdown status: na - task: lint_package_json status: na - task: lint_repl_help status: na - task: lint_javascript_src status: na - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: na - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: na - task: lint_c_tests_fixtures status: na - task: lint_shell status: na - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: na - task: lint_license_headers status: passed --- --- .../test/fixtures/start-location/file.js | 1 - .../@stdlib/assert/is-prototype-of/benchmark/benchmark.js | 2 +- .../@stdlib/utils/writable-property-names-in/lib/main.js | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js b/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js index c636d2ffecef..b4d11852b7fc 100644 --- a/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js +++ b/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js @@ -2,7 +2,6 @@ // License header should be at the top of the file within the first 5 lines. - /** * @license Apache-2.0 * diff --git a/lib/node_modules/@stdlib/assert/is-prototype-of/benchmark/benchmark.js b/lib/node_modules/@stdlib/assert/is-prototype-of/benchmark/benchmark.js index 801e512de42b..e4c91ec75280 100644 --- a/lib/node_modules/@stdlib/assert/is-prototype-of/benchmark/benchmark.js +++ b/lib/node_modules/@stdlib/assert/is-prototype-of/benchmark/benchmark.js @@ -40,7 +40,7 @@ bench( pkg, function benchmark( b ) { vals = [ new Date(), new RegExp( '.*' ), // eslint-disable-line prefer-regex-literals - new Array( 10 ), + new Array( 10 ), // eslint-disable-line stdlib/no-new-array new Foo() ]; b.tic(); diff --git a/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js b/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js index 959ec97d6d64..c9b86245607e 100644 --- a/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js +++ b/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js @@ -61,7 +61,7 @@ function writablePropertyNamesIn( value ) { return []; } // Cast the value to an object: - obj = Object( value ); + obj = Object( value ); // eslint-disable-line stdlib/require-globals // Walk the prototype chain collecting writable property names... names = []; From 688231ffb16cf0c000c1a62a72c5b340b232de1e Mon Sep 17 00:00:00 2001 From: Athan Date: Thu, 30 Jul 2026 15:30:43 -0700 Subject: [PATCH 2/3] Apply suggestions from code review Co-authored-by: Athan Signed-off-by: Athan --- .../test/fixtures/start-location/file.js | 3 ++- .../@stdlib/utils/writable-property-names-in/lib/main.js | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js b/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js index b4d11852b7fc..46bef11c0534 100644 --- a/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js +++ b/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js @@ -1,7 +1,8 @@ -/* eslint-ignore no-multiple-empty-lines */ +/* eslint-ignore stdlib/no-multiple-empty-lines */ // License header should be at the top of the file within the first 5 lines. + /** * @license Apache-2.0 * diff --git a/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js b/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js index c9b86245607e..dcbd2fe67a2b 100644 --- a/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js +++ b/lib/node_modules/@stdlib/utils/writable-property-names-in/lib/main.js @@ -24,6 +24,7 @@ var getOwnPropertyNames = require( '@stdlib/utils/property-names' ); var getPrototypeOf = require( '@stdlib/utils/get-prototype-of' ); var isWritableProperty = require( '@stdlib/assert/is-writable-property' ); var hasOwnProp = require( '@stdlib/assert/has-own-property' ); +var Object = require( '@stdlib/object/ctor' ); // MAIN // @@ -61,7 +62,7 @@ function writablePropertyNamesIn( value ) { return []; } // Cast the value to an object: - obj = Object( value ); // eslint-disable-line stdlib/require-globals + obj = Object( value ); // Walk the prototype chain collecting writable property names... names = []; From 91c0037c1de2beb3558fba79d96665c88271a828 Mon Sep 17 00:00:00 2001 From: Athan Date: Thu, 30 Jul 2026 15:34:00 -0700 Subject: [PATCH 3/3] Apply suggestions from code review Co-authored-by: Athan Signed-off-by: Athan --- .../test/fixtures/start-location/file.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js b/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js index 46bef11c0534..26cd8b6742d0 100644 --- a/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js +++ b/lib/node_modules/@stdlib/_tools/lint/license-header-file-list/test/fixtures/start-location/file.js @@ -1,4 +1,4 @@ -/* eslint-ignore stdlib/no-multiple-empty-lines */ +/* eslint-disable stdlib/no-multiple-empty-lines */ // License header should be at the top of the file within the first 5 lines.