mediawiki/extensions/ContentStabilization: main (log #1241798)

sourcepatches

This run took 27 seconds.

$ date
--- stdout ---
Tue Apr  9 03:18:18 UTC 2024

--- end ---
$ git clone file:///srv/git/mediawiki-extensions-ContentStabilization.git repo --depth=1 -b master
--- stderr ---
Cloning into 'repo'...
--- stdout ---

--- end ---
$ git config user.name libraryupgrader
--- stdout ---

--- end ---
$ git config user.email tools.libraryupgrader@tools.wmflabs.org
--- stdout ---

--- end ---
$ git submodule update --init
--- stdout ---

--- end ---
$ grr init
--- stdout ---
Installed commit-msg hook.

--- end ---
$ git show-ref refs/heads/master
--- stdout ---
382ab3e883248a21ca8b6718276104441008c3a9 refs/heads/master

--- end ---
$ /usr/bin/npm audit --json
--- stdout ---
{
  "auditReportVersion": 2,
  "vulnerabilities": {},
  "metadata": {
    "vulnerabilities": {
      "info": 0,
      "low": 0,
      "moderate": 0,
      "high": 0,
      "critical": 0,
      "total": 0
    },
    "dependencies": {
      "prod": 1,
      "dev": 313,
      "optional": 0,
      "peer": 1,
      "peerOptional": 0,
      "total": 313
    }
  }
}

--- end ---
$ /usr/bin/composer install
--- stderr ---
No composer.lock file present. Updating dependencies to latest instead of installing from lock file. See https://getcomposer.org/install for more information.
Loading composer repositories with package information
Updating dependencies
Lock file operations: 23 installs, 0 updates, 0 removals
  - Locking composer/installers (v2.2.0)
  - Locking composer/semver (3.3.2)
  - Locking composer/spdx-licenses (1.5.8)
  - Locking mediawiki/mediawiki-codesniffer (v41.0.0)
  - Locking mediawiki/minus-x (1.1.1)
  - Locking mwstake/mediawiki-component-events (2.0.8)
  - Locking mwstake/mediawiki-component-notifications (2.0.2)
  - Locking mwstake/mediawiki-componentloader (1.0.0)
  - Locking php-parallel-lint/php-console-color (v1.0.1)
  - Locking php-parallel-lint/php-console-highlighter (v1.0.0)
  - Locking php-parallel-lint/php-parallel-lint (v1.3.2)
  - Locking psr/container (2.0.2)
  - Locking squizlabs/php_codesniffer (3.7.2)
  - Locking symfony/console (v5.4.36)
  - Locking symfony/deprecation-contracts (v3.4.0)
  - Locking symfony/polyfill-ctype (v1.29.0)
  - Locking symfony/polyfill-intl-grapheme (v1.29.0)
  - Locking symfony/polyfill-intl-normalizer (v1.29.0)
  - Locking symfony/polyfill-mbstring (v1.29.0)
  - Locking symfony/polyfill-php73 (v1.29.0)
  - Locking symfony/polyfill-php80 (v1.29.0)
  - Locking symfony/service-contracts (v3.4.2)
  - Locking symfony/string (v6.4.4)
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 23 installs, 0 updates, 0 removals
    0 [>---------------------------]    0 [->--------------------------]
  - Installing composer/installers (v2.2.0): Extracting archive
  - Installing symfony/polyfill-php80 (v1.29.0): Extracting archive
  - Installing squizlabs/php_codesniffer (3.7.2): Extracting archive
  - Installing symfony/polyfill-mbstring (v1.29.0): Extracting archive
  - Installing composer/spdx-licenses (1.5.8): Extracting archive
  - Installing composer/semver (3.3.2): Extracting archive
  - Installing mediawiki/mediawiki-codesniffer (v41.0.0): Extracting archive
  - Installing symfony/polyfill-intl-normalizer (v1.29.0): Extracting archive
  - Installing symfony/polyfill-intl-grapheme (v1.29.0): Extracting archive
  - Installing symfony/polyfill-ctype (v1.29.0): Extracting archive
  - Installing symfony/string (v6.4.4): Extracting archive
  - Installing psr/container (2.0.2): Extracting archive
  - Installing symfony/service-contracts (v3.4.2): Extracting archive
  - Installing symfony/polyfill-php73 (v1.29.0): Extracting archive
  - Installing symfony/deprecation-contracts (v3.4.0): Extracting archive
  - Installing symfony/console (v5.4.36): Extracting archive
  - Installing mediawiki/minus-x (1.1.1): Extracting archive
  - Installing mwstake/mediawiki-componentloader (1.0.0): Extracting archive
  - Installing mwstake/mediawiki-component-events (2.0.8): Extracting archive
  - Installing mwstake/mediawiki-component-notifications (2.0.2): Extracting archive
  - Installing php-parallel-lint/php-console-color (v1.0.1): Extracting archive
  - Installing php-parallel-lint/php-console-highlighter (v1.0.0): Extracting archive
  - Installing php-parallel-lint/php-parallel-lint (v1.3.2): Extracting archive
  0/22 [>---------------------------]   0%
 21/22 [==========================>-]  95%
 22/22 [============================] 100%
4 package suggestions were added by new dependencies, use `composer suggest` to see details.
Generating autoload files
14 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
--- stdout ---

--- end ---
Upgrading n:grunt-banana-checker from 0.10.0 -> 0.11.1
$ /usr/bin/npm install
--- stdout ---

added 313 packages, and audited 314 packages in 3s

55 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

--- end ---
$ package-lock-lint package-lock.json
--- stdout ---
Checking package-lock.json

--- end ---
$ /usr/bin/npm ci
--- stdout ---

added 313 packages, and audited 314 packages in 4s

55 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

--- end ---
$ /usr/bin/npm test
--- stdout ---

> test
> grunt test

Running "eslint:all" (eslint) task

/src/repo/resources/ext.contentStabilization.alert.js
   4:18  warning  Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible  no-jquery/no-global-selector
   9:3   warning  Combine this with the previous 'const' statement                                    one-var
  11:2   warning  Mixed spaces and tabs                                                               no-mixed-spaces-and-tabs
  21:3   warning  Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible  no-jquery/no-global-selector
  38:3   warning  Combine this with the previous 'const' statement                                    one-var
  40:15  warning  Do not access Object.prototype method 'hasOwnProperty' from target object           no-prototype-builtins

/src/repo/resources/ext.contentStabilization.bootstrap.js
   7:10  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  13:10  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  19:10  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  23:11  warning  Unexpected dangling '_' in '_ajax'                                                  no-underscore-dangle
  23:11  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  26:11  warning  Unexpected dangling '_' in '_ajax'                                                  no-underscore-dangle
  26:11  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  29:11  warning  Unexpected dangling '_' in '_ajax'                                                  no-underscore-dangle
  29:11  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  34:8   warning  'dfd' is never reassigned. Use 'const' instead                                      prefer-const
  40:4   warning  Unexpected dangling '_' in '_requests'                                              no-underscore-dangle
  40:4   warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  47:11  warning  Unexpected dangling '_' in '_requests'                                              no-underscore-dangle
  47:11  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  47:51  warning  Do not access Object.prototype method 'hasOwnProperty' from target object           no-prototype-builtins
  48:7   warning  Unexpected dangling '_' in '_requests'                                              no-underscore-dangle
  48:7   warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  52:14  warning  Unexpected dangling '_' in '_requests'                                              no-underscore-dangle
  52:14  warning  Unexpected dangling '_' in '_api'                                                   no-underscore-dangle
  55:14  warning  Unexpected dangling '_' in '_requests'                                              no-underscore-dangle
  81:4   warning  Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible  no-jquery/no-global-selector

/src/repo/resources/ext.contentStabilization.enhancedHistory.js
   6:19  warning  Identifier 'sp_state' is not in camel case            camelcase
  13:19  warning  Identifier 'sp_approver' is not in camel case         camelcase
  21:19  warning  Identifier 'sp_approve_ts' is not in camel case       camelcase
  28:19  warning  Identifier 'sp_approve_comment' is not in camel case  camelcase

/src/repo/resources/ext.contentStabilization.special.js
    2:15  warning  Avoid queries which search the entire DOM. Keep DOM nodes in memory where possible  no-jquery/no-global-selector
    6:2   warning  Combine this with the previous 'const' statement                                    one-var
    6:20  warning  'OOJSPlus' is not defined                                                           no-undef
   11:4   warning  Identifier 'page_display_text' is not in camel case                                 camelcase
   35:4   warning  Identifier 'has_changed_inclusions' is not in camel case                            camelcase
   38:36  warning  'row' is defined but never used                                                     no-unused-vars
   42:4   warning  Identifier 'last_approver' is not in camel case                                     camelcase
   49:4   warning  Identifier 'last_stable_ts' is not in camel case                                    camelcase
   55:4   warning  Identifier 'last_comment' is not in camel case                                      camelcase
   60:14  warning  'OOJSPlus' is not defined                                                           no-undef
   63:5   warning  Identifier 'page_title' is not in camel case                                        camelcase
   70:17  warning  'OOJSPlus' is not defined                                                           no-undef
   74:7   warning  Identifier 'page_title' is not in camel case                                        camelcase
   80:9   warning  '$table' is never reassigned. Use 'const' instead                                   prefer-const
  116:21  warning  Do not access Object.prototype method 'hasOwnProperty' from target object           no-prototype-builtins

/src/repo/resources/pageassignments.addStabilizationColumn.js
   1:93  warning  'actions' is defined but never used   no-unused-vars
  13:31  warning  'metaData' is defined but never used  no-unused-vars
  13:41  warning  'record' is defined but never used    no-unused-vars
  13:49  warning  'rowIndex' is defined but never used  no-unused-vars
  13:59  warning  'colIndex' is defined but never used  no-unused-vars
  13:69  warning  'store' is defined but never used     no-unused-vars
  13:76  warning  'view' is defined but never used      no-unused-vars

/src/repo/resources/ui/ApproveDialog.js
  64:8  warning  Where possible, maintain application state in JS to avoid slower DOM queries  no-jquery/no-class-state

/src/repo/resources/ui/workflows/inspector/ApprovePageInspector.js
   2:1    warning  This line has a length of 111. Maximum allowed is 100  max-len
   4:80   warning  Missing space before function parentheses              space-before-function-paren
   5:1    warning  This line has a length of 110. Maximum allowed is 100  max-len
   8:1    warning  This line has a length of 134. Maximum allowed is 100  max-len
   8:88   warning  'workflows' is not defined                             no-undef
  10:1    warning  This line has a length of 108. Maximum allowed is 100  max-len
  10:105  warning  Missing space before function parentheses              space-before-function-paren
  14:1    warning  This line has a length of 102. Maximum allowed is 100  max-len
  14:99   warning  Missing space before function parentheses              space-before-function-paren
  15:8    warning  Multiple spaces found before '['                       no-multi-spaces
  40:1    warning  'workflows' is not defined                             no-undef

/src/repo/resources/ui/workflows/registerCustomElements.js
  1:1  warning  'workflows' is not defined  no-undef

✖ 66 problems (0 errors, 66 warnings)
  0 errors and 4 warnings potentially fixable with the `--fix` option.


Running "banana:ContentStabilization" (banana) task
>> 1 documented message is blank.
>> Message "content-stabilization-integration-workflows-property-validator-valid-reviewer-error" is documented with a blank string.
>> 1 message has trailing whitespace:
>> Message "contentstabilization-export-laststable-tag-text" has trailing whitespace
>> The "ar" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "bg" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "bn" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "de" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "el" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "fi" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "fr" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "he" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "hi" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "ia" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "ja" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "ko" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "lb" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "mk" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "nl" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "skr-arab" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "sl" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "sr-ec" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "sv" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "tr" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "wal" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
>> The "zh-hant" translation has 1 translation with trailing whitespace:
>> The translation of "contentstabilization-export-laststable-tag-text" has trailing whitespace.
Warning: Task "banana:ContentStabilization" failed. Use --force to continue.

Aborted due to warnings.

--- end ---
Traceback (most recent call last):
  File "/venv/lib/python3.11/site-packages/runner-0.1.0-py3.11.egg/runner/__init__.py", line 1534, in main
    libup.run(args.repo, args.output, args.branch)
  File "/venv/lib/python3.11/site-packages/runner-0.1.0-py3.11.egg/runner/__init__.py", line 1472, in run
    self.npm_upgrade(plan)
  File "/venv/lib/python3.11/site-packages/runner-0.1.0-py3.11.egg/runner/__init__.py", line 1057, in npm_upgrade
    self.npm_test()
  File "/venv/lib/python3.11/site-packages/runner-0.1.0-py3.11.egg/runner/__init__.py", line 297, in npm_test
    self.check_call(['npm', 'test'])
  File "/venv/lib/python3.11/site-packages/runner-0.1.0-py3.11.egg/runner/shell2.py", line 54, in check_call
    res.check_returncode()
  File "/usr/lib/python3.11/subprocess.py", line 502, in check_returncode
    raise CalledProcessError(self.returncode, self.args, self.stdout,
subprocess.CalledProcessError: Command '['/usr/bin/npm', 'test']' returned non-zero exit status 3.
Source code is licensed under the AGPL.