Skip to content
Validate Monorepo

fix(commonjs): correctly replaces shorthand "global" property in object #2096

Sign in to view logs

fix(commonjs): correctly replaces shorthand "global" property in object

fix(commonjs): correctly replaces shorthand "global" property in object #2096

Triggered via pull request January 26, 2026 05:56
@TrickyPiTrickyPi
edited #1957
TrickyPi:fix/handle-shorthand-global
Status Success
Total duration 1m 43s
Artifacts –

validate.yml

on: pull_request
Matrix: build
Fit to window
Zoom out
Zoom in

Annotations

20 warnings
Node v18: packages/node-resolve/test/prefer-builtins.js#L158
Unexpected require()
Node v18: packages/node-resolve/test/prefer-builtins.js#L157
Unexpected require()
Node v18: packages/node-resolve/src/package/resolvePackageTarget.ts#L56
Assignment to function parameter 'target'
Node v18: packages/node-resolve/src/index.js#L182
'exportConditions' is already declared in the upper scope on line 55 column 9
Node v18: packages/eslint/test/test.mjs#L251
'eslint' is already declared in the upper scope on line 13 column 8
Node v18: packages/eslint/test/test.mjs#L251
'eslint' is already declared in the upper scope on line 13 column 8
Node v18: packages/eslint/src/index.ts#L11
'loadESLint' is already declared in the upper scope on line 5 column 18
Node v18: packages/commonjs/test/test-dynamic-require-root-outside-cwd.js#L21
Use array destructuring
Node v18: packages/commonjs/test/helpers/util.js#L10
'path' is already declared in the upper scope on line 1 column 7
Node v18: packages/commonjs/test/helpers/util.js#L10
'path' is already declared in the upper scope on line 1 column 7
Node v20: packages/node-resolve/test/prefer-builtins.js#L158
Unexpected require()
Node v20: packages/node-resolve/test/prefer-builtins.js#L157
Unexpected require()
Node v20: packages/node-resolve/src/package/resolvePackageTarget.ts#L56
Assignment to function parameter 'target'
Node v20: packages/node-resolve/src/index.js#L182
'exportConditions' is already declared in the upper scope on line 55 column 9
Node v20: packages/eslint/test/test.mjs#L251
'eslint' is already declared in the upper scope on line 13 column 8
Node v20: packages/eslint/test/test.mjs#L251
'eslint' is already declared in the upper scope on line 13 column 8
Node v20: packages/eslint/src/index.ts#L11
'loadESLint' is already declared in the upper scope on line 5 column 18
Node v20: packages/commonjs/test/test-dynamic-require-root-outside-cwd.js#L21
Use array destructuring
Node v20: packages/commonjs/test/helpers/util.js#L10
'path' is already declared in the upper scope on line 1 column 7
Node v20: packages/commonjs/test/helpers/util.js#L10
'path' is already declared in the upper scope on line 1 column 7