diff --git a/.github/files/build-reminder-comment/check-test-reminder-comment.js b/.github/files/build-reminder-comment/check-test-reminder-comment.js index 1bc2785df57..69a9eef212b 100644 --- a/.github/files/build-reminder-comment/check-test-reminder-comment.js +++ b/.github/files/build-reminder-comment/check-test-reminder-comment.js @@ -33,7 +33,7 @@ async function isTouchingSomethingNeedingTesting( github, owner, repo, number, c * If there isn't, create one if it is needed. * * @param {github} github - Pre-authenticated octokit/rest.js client with pagination plugins - * @param {Object} context - Context of the workflow run + * @param {object} context - Context of the workflow run * @param {core} core - A reference to the @actions/core package * @returns {Promise} Promise resolving to a comment ID, or 0 if no comment is found. */ diff --git a/.pnpmfile.cjs b/.pnpmfile.cjs index 6409cabfab4..24bfe1ce869 100644 --- a/.pnpmfile.cjs +++ b/.pnpmfile.cjs @@ -156,6 +156,14 @@ function fixPeerDeps( pkg ) { pkg.peerDependencies[ '@wordpress/data' ] = '^6.1.5 || ^7.0.0'; } + // Outdated peer dependency. + if ( + pkg.name === 'eslint-plugin-wpcalypso' && + pkg.peerDependencies[ 'eslint-plugin-jsdoc' ] === '^37.5.1' + ) { + pkg.peerDependencies[ 'eslint-plugin-jsdoc' ] = '>=37.5.1'; + } + return pkg; } diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index 5cfbbdd285a..13580c08621 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -920,6 +920,9 @@ importers: webpack: 5.72.1_webpack-cli@4.9.1 webpack-cli: 4.9.1_webpack@5.72.1 + projects/packages/forms: + specifiers: {} + projects/packages/google-fonts-provider: specifiers: {} @@ -2308,16 +2311,16 @@ importers: eslint-plugin-es: 4.1.0 eslint-plugin-import: 2.26.0 eslint-plugin-inclusive-language: 2.2.0 - eslint-plugin-jest: 26.8.7 + eslint-plugin-jest: 27.2.1 eslint-plugin-jest-dom: 4.0.2 - eslint-plugin-jsdoc: 37.9.7 + eslint-plugin-jsdoc: 39.6.6 eslint-plugin-jsx-a11y: 6.6.1 eslint-plugin-lodash: 7.4.0 eslint-plugin-playwright: 0.11.1 eslint-plugin-prettier: 4.2.1 eslint-plugin-react: 7.31.0 eslint-plugin-react-hooks: 4.5.0 - eslint-plugin-svelte3: 3.4.1 + eslint-plugin-svelte3: 4.0.0 eslint-plugin-testing-library: 5.6.0 eslint-plugin-wpcalypso: 6.1.0 glob: 7.1.6 @@ -2346,7 +2349,7 @@ importers: '@testing-library/jest-dom': 5.16.5 '@typescript-eslint/eslint-plugin': 5.36.1_wa5wfrbhchn2iw6bswuemn7pui '@typescript-eslint/parser': 5.36.1_shit3uhl6a7megkzgoz6xssnfa - '@wordpress/eslint-plugin': 13.6.0_jhgvbdj6afcuhsnzaiih2wciue + '@wordpress/eslint-plugin': 13.6.0_jxqbym6i2eh45w5ogbv2yfg3la '@wordpress/jest-console': 6.5.0_jest@29.3.1 babel-jest: 29.3.1_@babel+core@7.20.7 chalk: 4.1.2 @@ -2356,18 +2359,18 @@ importers: eslint-plugin-es: 4.1.0_eslint@8.22.0 eslint-plugin-import: 2.26.0_pwovp45kjytedpuwpq6ialv6fq eslint-plugin-inclusive-language: 2.2.0 - eslint-plugin-jest: 26.8.7_spapuu346vwpqebkax3iyiuevu + eslint-plugin-jest: 27.2.1_spapuu346vwpqebkax3iyiuevu eslint-plugin-jest-dom: 4.0.2_eslint@8.22.0 - eslint-plugin-jsdoc: 37.9.7_eslint@8.22.0 + eslint-plugin-jsdoc: 39.6.6_eslint@8.22.0 eslint-plugin-jsx-a11y: 6.6.1_eslint@8.22.0 eslint-plugin-lodash: 7.4.0_eslint@8.22.0 - eslint-plugin-playwright: 0.11.1_dsqeopb3eheqx4cl4z3tzeketi + eslint-plugin-playwright: 0.11.1_ascjnguiwqbin4dlvo6za4uvbq eslint-plugin-prettier: 4.2.1_dimnjh7sfkcndv2hbstnhh33ju eslint-plugin-react: 7.31.0_eslint@8.22.0 eslint-plugin-react-hooks: 4.5.0_eslint@8.22.0 - eslint-plugin-svelte3: 3.4.1_laaqauvsmoyypsiqkozwyi2fn4 + eslint-plugin-svelte3: 4.0.0_laaqauvsmoyypsiqkozwyi2fn4 eslint-plugin-testing-library: 5.6.0_shit3uhl6a7megkzgoz6xssnfa - eslint-plugin-wpcalypso: 6.1.0_v3ycduczdyau2ur5nkc365ajme + eslint-plugin-wpcalypso: 6.1.0_bxtg6lay27vkulnt4lmqoui7ce glob: 7.1.6 ignore: 5.1.8 inquirer: 7.3.3 @@ -5267,13 +5270,13 @@ packages: /@emotion/weak-memoize/0.3.0: resolution: {integrity: sha512-AHPmaAx+RYfZz0eYu6Gviiagpmiyw98ySSlQvCUhVGDRtDFe4DBS0x1bSjdF3gqUDYOczB+yYvBTtEylYSdRhg==} - /@es-joy/jsdoccomment/0.20.1: - resolution: {integrity: sha512-oeJK41dcdqkvdZy/HctKklJNkt/jh+av3PZARrZEl+fs/8HaHeeYoAvEwOV0u5I6bArTF17JEsTZMY359e/nfQ==} - engines: {node: ^12 || ^14 || ^16 || ^17} + /@es-joy/jsdoccomment/0.36.1: + resolution: {integrity: sha512-922xqFsTpHs6D0BUiG4toiyPOMc8/jafnWKxz1KWgS4XzKPy2qXf1Pe6UFuNSCQqt6tOuhAWXBNuuyUhJmw9Vg==} + engines: {node: ^14 || ^16 || ^17 || ^18 || ^19} dependencies: - comment-parser: 1.3.0 + comment-parser: 1.3.1 esquery: 1.4.0 - jsdoc-type-pratt-parser: 2.2.5 + jsdoc-type-pratt-parser: 3.1.0 dev: true /@esbuild/android-arm/0.15.18: @@ -8959,7 +8962,7 @@ packages: '@wordpress/dom': 3.23.0 '@wordpress/element': 4.20.0 '@wordpress/hooks': 3.23.0 - '@wordpress/html-entities': 3.24.0 + '@wordpress/html-entities': 3.22.0 '@wordpress/i18n': 4.23.0 '@wordpress/icons': 9.13.0 '@wordpress/is-shallow-equal': 4.23.0 @@ -9580,7 +9583,7 @@ packages: '@wordpress/dom': 3.23.0 '@wordpress/element': 4.20.0 '@wordpress/hooks': 3.23.0 - '@wordpress/html-entities': 3.24.0 + '@wordpress/html-entities': 3.22.0 '@wordpress/i18n': 4.23.0 '@wordpress/icons': 9.13.0 '@wordpress/keyboard-shortcuts': 3.20.0_react@17.0.2 @@ -9633,7 +9636,7 @@ packages: dependencies: '@babel/runtime': 7.20.7 - /@wordpress/eslint-plugin/13.6.0_jhgvbdj6afcuhsnzaiih2wciue: + /@wordpress/eslint-plugin/13.6.0_jxqbym6i2eh45w5ogbv2yfg3la: resolution: {integrity: sha512-GIW4AHb0IC9VA6y8IRGQpADxpvdG+K0an/ZpzlYmBudql7YuLlUmp9rLpBlSvHf9iGX81OUYl9B63XEItPzGEw==} engines: {node: '>=14', npm: '>=6.14.4'} peerDependencies: @@ -9665,8 +9668,8 @@ packages: eslint: 8.22.0 eslint-config-prettier: 8.5.0_eslint@8.22.0 eslint-plugin-import: 2.26.0_pwovp45kjytedpuwpq6ialv6fq - eslint-plugin-jest: 26.8.7_spapuu346vwpqebkax3iyiuevu - eslint-plugin-jsdoc: 37.9.7_eslint@8.22.0 + eslint-plugin-jest: 27.2.1_spapuu346vwpqebkax3iyiuevu + eslint-plugin-jsdoc: 39.6.6_eslint@8.22.0 eslint-plugin-jsx-a11y: 6.6.1_eslint@8.22.0 eslint-plugin-prettier: 4.2.1_dimnjh7sfkcndv2hbstnhh33ju eslint-plugin-react: 7.31.0_eslint@8.22.0 @@ -11977,8 +11980,8 @@ packages: engines: {node: ^12.20.0 || >=14} dev: true - /comment-parser/1.3.0: - resolution: {integrity: sha512-hRpmWIKgzd81vn0ydoWoyPoALEOnF4wt8yKD35Ib1D6XC2siLiYaiqfGkYrunuKdsXGwpBpHU3+9r+RVw2NZfA==} + /comment-parser/1.3.1: + resolution: {integrity: sha512-B52sN2VNghyq5ofvUsqZjmk6YkihBX5vMSChmSK9v4ShjKf3Vk5Xcmgpw4o+iIgtrnM/u5FiMpz9VKb8lpBveA==} engines: {node: '>= 12.0.0'} dev: true @@ -13611,12 +13614,12 @@ packages: requireindex: 1.2.0 dev: true - /eslint-plugin-jest/26.8.7_spapuu346vwpqebkax3iyiuevu: - resolution: {integrity: sha512-nJJVv3VY6ZZvJGDMC8h1jN/TIGT4We1JkNn1lvstPURicr/eZPVnlFULQ4W2qL9ByCuCr1hPmlBOc2aZ1ktw4Q==} - engines: {node: ^12.22.0 || ^14.17.0 || >=16.0.0} + /eslint-plugin-jest/27.2.1_spapuu346vwpqebkax3iyiuevu: + resolution: {integrity: sha512-l067Uxx7ZT8cO9NJuf+eJHvt6bqJyz2Z29wykyEdz/OtmcELQl2MQGQLX8J94O1cSJWAwUSEvCjwjA7KEK3Hmg==} + engines: {node: ^14.15.0 || ^16.10.0 || >=18.0.0} peerDependencies: '@typescript-eslint/eslint-plugin': ^5.0.0 - eslint: ^6.0.0 || ^7.0.0 || ^8.0.0 + eslint: ^7.0.0 || ^8.0.0 jest: '*' peerDependenciesMeta: '@typescript-eslint/eslint-plugin': @@ -13633,19 +13636,18 @@ packages: - typescript dev: true - /eslint-plugin-jsdoc/37.9.7_eslint@8.22.0: - resolution: {integrity: sha512-8alON8yYcStY94o0HycU2zkLKQdcS+qhhOUNQpfONHHwvI99afbmfpYuPqf6PbLz5pLZldG3Te5I0RbAiTN42g==} - engines: {node: ^12 || ^14 || ^16 || ^17} + /eslint-plugin-jsdoc/39.6.6_eslint@8.22.0: + resolution: {integrity: sha512-cFkmcdMYHw4XwA4fM5Rdopn9wB2p+Yu966F/6fa9C/oIMH99+YCfhH3+qXMc39VX9KYZ0d/KHgUbKo8D7N/27A==} + engines: {node: ^14 || ^16 || ^17 || ^18 || ^19} peerDependencies: eslint: ^7.0.0 || ^8.0.0 dependencies: - '@es-joy/jsdoccomment': 0.20.1 - comment-parser: 1.3.0 + '@es-joy/jsdoccomment': 0.36.1 + comment-parser: 1.3.1 debug: 4.3.4 escape-string-regexp: 4.0.0 eslint: 8.22.0 esquery: 1.4.0 - regextras: 0.8.0 semver: 7.3.8 spdx-expression-parse: 3.0.1 transitivePeerDependencies: @@ -13684,7 +13686,7 @@ packages: lodash: 4.17.21 dev: true - /eslint-plugin-playwright/0.11.1_dsqeopb3eheqx4cl4z3tzeketi: + /eslint-plugin-playwright/0.11.1_ascjnguiwqbin4dlvo6za4uvbq: resolution: {integrity: sha512-SFoz9Sh+FCUPaFrM2iEJ0gkQhx8pT8rNcVFZUsRJEhTxRigK1HHs7dQi30h1QOIoPHt3hpdFVslBPGfEY/mogQ==} peerDependencies: eslint: '>=7' @@ -13694,7 +13696,7 @@ packages: optional: true dependencies: eslint: 8.22.0 - eslint-plugin-jest: 26.8.7_spapuu346vwpqebkax3iyiuevu + eslint-plugin-jest: 27.2.1_spapuu346vwpqebkax3iyiuevu dev: true /eslint-plugin-prettier/4.2.1_dimnjh7sfkcndv2hbstnhh33ju: @@ -13746,11 +13748,10 @@ packages: string.prototype.matchall: 4.0.8 dev: true - /eslint-plugin-svelte3/3.4.1_laaqauvsmoyypsiqkozwyi2fn4: - resolution: {integrity: sha512-7p59WG8qV8L6wLdl4d/c3mdjkgVglQCdv5XOTk/iNPBKXuuV+Q0eFP5Wa6iJd/G2M1qR3BkLPEzaANOqKAZczw==} - engines: {node: '>=10'} + /eslint-plugin-svelte3/4.0.0_laaqauvsmoyypsiqkozwyi2fn4: + resolution: {integrity: sha512-OIx9lgaNzD02+MDFNLw0GEUbuovNcglg+wnd/UY0fbZmlQSz7GlQiQ1f+yX0XvC07XPcDOnFcichqI3xCwp71g==} peerDependencies: - eslint: '>=6.0.0' + eslint: '>=8.0.0' svelte: ^3.2.0 dependencies: eslint: 8.22.0 @@ -13770,14 +13771,14 @@ packages: - typescript dev: true - /eslint-plugin-wpcalypso/6.1.0_v3ycduczdyau2ur5nkc365ajme: + /eslint-plugin-wpcalypso/6.1.0_bxtg6lay27vkulnt4lmqoui7ce: resolution: {integrity: sha512-T/tetEGvrmSJ0lqNL7jVMmApTWEtMYR334vc3kFt3kdcBhsHnIDrdJYp9+wabKIV0hxv21tujLnD+BH/MzWeGQ==} engines: {node: '>=10'} peerDependencies: '@babel/core': '>=7.17.5' eslint: '>=8.6.0' eslint-plugin-inclusive-language: ^2.2.0 - eslint-plugin-jsdoc: ^37.5.1 + eslint-plugin-jsdoc: '>=37.5.1' eslint-plugin-react: ^7.28.0 eslint-plugin-react-hooks: ^4.3.0 peerDependenciesMeta: @@ -13789,7 +13790,7 @@ packages: '@babel/core': 7.20.7 eslint: 8.22.0 eslint-plugin-inclusive-language: 2.2.0 - eslint-plugin-jsdoc: 37.9.7_eslint@8.22.0 + eslint-plugin-jsdoc: 39.6.6_eslint@8.22.0 eslint-plugin-react: 7.31.0_eslint@8.22.0 eslint-plugin-react-hooks: 4.5.0_eslint@8.22.0 dev: true @@ -16968,8 +16969,8 @@ packages: dependencies: argparse: 2.0.1 - /jsdoc-type-pratt-parser/2.2.5: - resolution: {integrity: sha512-2a6eRxSxp1BW040hFvaJxhsCMI9lT8QB8t14t+NY5tC5rckIR0U9cr2tjOeaFirmEOy6MHvmJnY7zTBHq431Lw==} + /jsdoc-type-pratt-parser/3.1.0: + resolution: {integrity: sha512-MgtD0ZiCDk9B+eI73BextfRrVQl0oyzRG8B2BjORts6jbunj4ScKPcyXGTbB6eXL4y9TzxCm6hyeLq/2ASzNdw==} engines: {node: '>=12.0.0'} dev: true @@ -20298,11 +20299,6 @@ packages: unicode-match-property-ecmascript: 2.0.0 unicode-match-property-value-ecmascript: 2.1.0 - /regextras/0.8.0: - resolution: {integrity: sha512-k519uI04Z3SaY0fLX843MRXnDeG2+vHOFsyhiPZvNLe7r8rD2YNRjq4BQLZZ0oAr2NrtvZlICsXysGNFPGa3CQ==} - engines: {node: '>=0.1.14'} - dev: true - /regjsgen/0.7.1: resolution: {integrity: sha512-RAt+8H2ZEzHeYWxZ3H2z6tF18zyyOnlcdaafLrm21Bguj7uZy6ULibiAFdXEtKQY4Sy7wDTwDiOazasMLc4KPA==} diff --git a/projects/github-actions/repo-gardening/changelog/renovate-major-eslint-packages b/projects/github-actions/repo-gardening/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/github-actions/repo-gardening/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/github-actions/repo-gardening/src/tasks/gather-support-references/index.js b/projects/github-actions/repo-gardening/src/tasks/gather-support-references/index.js index d2d7b556efe..089429364eb 100644 --- a/projects/github-actions/repo-gardening/src/tasks/gather-support-references/index.js +++ b/projects/github-actions/repo-gardening/src/tasks/gather-support-references/index.js @@ -10,7 +10,7 @@ const sendSlackMessage = require( '../../utils/send-slack-message' ); * Search for a previous comment from this task in our issue. * * @param {Array} issueComments - Array of all comments on that issue. - * @returns {Promise} Promise resolving to an object of information about our comment. + * @returns {Promise} Promise resolving to an object of information about our comment. */ async function getListComment( issueComments ) { let commentInfo = {}; @@ -129,7 +129,7 @@ ${ issueReferences * @param {WebhookPayloadIssue} payload - Issue event payload. * @param {string} channel - Slack channel ID. * @param {string} message - Basic message (without the formatting). - * @returns {Object} Object containing the slack message and its formatting. + * @returns {object} Object containing the slack message and its formatting. */ function formatSlackMessage( payload, channel, message ) { const { issue, repository } = payload; diff --git a/projects/github-actions/repo-gardening/src/tasks/reply-to-customers-reminder/index.js b/projects/github-actions/repo-gardening/src/tasks/reply-to-customers-reminder/index.js index ae9bc6f9ad0..2cfd9cda4ea 100644 --- a/projects/github-actions/repo-gardening/src/tasks/reply-to-customers-reminder/index.js +++ b/projects/github-actions/repo-gardening/src/tasks/reply-to-customers-reminder/index.js @@ -56,7 +56,7 @@ async function hasManySupportReferences( issueComments ) { * @param {WebhookPayloadIssue} payload - Issue event payload. * @param {string} channel - Slack channel ID. * @param {string} message - Basic message (without the formatting). - * @returns {Object} Object containing the slack message and its formatting. + * @returns {object} Object containing the slack message and its formatting. */ function formatSlackMessage( payload, channel, message ) { const { issue, repository } = payload; diff --git a/projects/github-actions/repo-gardening/src/utils/send-slack-message.js b/projects/github-actions/repo-gardening/src/utils/send-slack-message.js index 67426ef1423..57ab727ee3b 100644 --- a/projects/github-actions/repo-gardening/src/utils/send-slack-message.js +++ b/projects/github-actions/repo-gardening/src/utils/send-slack-message.js @@ -9,7 +9,7 @@ const fetch = require( 'node-fetch' ); * @param {string} channel - Slack channel ID. * @param {string} token - Slack token. * @param {WebhookPayloadPullRequest} payload - Pull request event payload. - * @param {Object} customMessageFormat - Custom message formatting. If defined, takes over from message completely. + * @param {object} customMessageFormat - Custom message formatting. If defined, takes over from message completely. * @returns {Promise} Promise resolving to a boolean, whether message was successfully posted or not. */ async function sendSlackMessage( message, channel, token, payload, customMessageFormat = {} ) { diff --git a/projects/github-actions/test-results-to-slack/changelog/renovate-major-eslint-packages b/projects/github-actions/test-results-to-slack/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/github-actions/test-results-to-slack/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/github-actions/test-results-to-slack/package.json b/projects/github-actions/test-results-to-slack/package.json index b1fc53e5a83..b3a755986ab 100644 --- a/projects/github-actions/test-results-to-slack/package.json +++ b/projects/github-actions/test-results-to-slack/package.json @@ -1,6 +1,6 @@ { "name": "test-results-to-slack", - "version": "0.2.0", + "version": "0.2.1-alpha", "description": "GitHub Action to send Slack notifications with test results", "author": "Automattic", "license": "GPL-2.0-or-later", diff --git a/projects/github-actions/test-results-to-slack/src/slack.js b/projects/github-actions/test-results-to-slack/src/slack.js index f2ee1372630..4ec741e4f99 100644 --- a/projects/github-actions/test-results-to-slack/src/slack.js +++ b/projects/github-actions/test-results-to-slack/src/slack.js @@ -4,9 +4,9 @@ const { debug, error } = require( './debug' ); /** * Sends a Slack message. * - * @param {Object} client - Slack client + * @param {object} client - Slack client * @param {boolean} update - if it should update a message. For true, it will update an existing message based on `ts`, false will send a new message. - * @param {Object} options - options + * @param {object} options - options * @returns {Promise<*>} the response from the Slack API. In case when multiple messages are sent due to the blocks length the last message response is returned. */ async function postOrUpdateMessage( client, update, options ) { @@ -129,7 +129,7 @@ function getBlocksChunks( blocks, maxSize, typeDelimiter ) { /** * Finds and returns a Slack message that contains a given string in its text (not in blocks!) * - * @param {Object} client - the Slack client + * @param {object} client - the Slack client * @param {string} channelId - the channel id * @param {string} identifier - the string to search for in the messages text * @returns {Promise<*|null>} the message Object diff --git a/projects/github-actions/test-results-to-slack/tests/rules.test.js b/projects/github-actions/test-results-to-slack/tests/rules.test.js index c07a7d93c1b..2e7c6d3fc62 100644 --- a/projects/github-actions/test-results-to-slack/tests/rules.test.js +++ b/projects/github-actions/test-results-to-slack/tests/rules.test.js @@ -43,7 +43,7 @@ describe( 'Notification rules', () => { * Writes the rules object to a temp file and returns the path to that file. * * @param {object} rules - the rules object - * @returns {string} the path to the temp file + * @returns {string|undefined} the path to the temp file */ function writeRules( rules ) { const rulesPath = './tests/ignore'; @@ -57,4 +57,6 @@ function writeRules( rules ) { fs.writeFileSync( rulePath, JSON.stringify( rules ) ); return path.resolve( rulePath ); } + + return undefined; } diff --git a/projects/js-packages/connection/changelog/renovate-major-eslint-packages b/projects/js-packages/connection/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/js-packages/connection/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/js-packages/connection/components/disconnect-dialog/index.jsx b/projects/js-packages/connection/components/disconnect-dialog/index.jsx index bef38f8cd3d..0a7d00d3e3d 100644 --- a/projects/js-packages/connection/components/disconnect-dialog/index.jsx +++ b/projects/js-packages/connection/components/disconnect-dialog/index.jsx @@ -302,7 +302,7 @@ const DisconnectDialog = props => { /** * Determine what step to show based on the current state * - * @returns { React.Component } - component for current step + * @returns { React.Component|undefined } - component for current step */ const getCurrentStep = () => { if ( ! isDisconnected ) { @@ -342,6 +342,8 @@ const DisconnectDialog = props => { } else if ( isFeedbackProvided ) { return ; } + + return undefined; }; return ( diff --git a/projects/js-packages/connection/components/disconnect-dialog/steps/step-disconnect.jsx b/projects/js-packages/connection/components/disconnect-dialog/steps/step-disconnect.jsx index e677d765587..31e37acaae4 100644 --- a/projects/js-packages/connection/components/disconnect-dialog/steps/step-disconnect.jsx +++ b/projects/js-packages/connection/components/disconnect-dialog/steps/step-disconnect.jsx @@ -93,7 +93,7 @@ const StepDisconnect = props => { * Show some fallback output if there are no connected plugins to show and no passed disconnect component. * This is a more generic message about disconnecting Jetpack. * - * @returns {React.ElementType} - Fallback message for when there are no connected plugins or passed components to show. + * @returns {React.ElementType|undefined} - Fallback message for when there are no connected plugins or passed components to show. */ const renderFallbackOutput = () => { const hasOtherConnectedPlugins = @@ -111,6 +111,8 @@ const StepDisconnect = props => { ); } + + return undefined; }; return ( diff --git a/projects/js-packages/connection/hooks/use-connection-error-notice/index.jsx b/projects/js-packages/connection/hooks/use-connection-error-notice/index.jsx index 92d7ad6a5c4..aae409c9abc 100644 --- a/projects/js-packages/connection/hooks/use-connection-error-notice/index.jsx +++ b/projects/js-packages/connection/hooks/use-connection-error-notice/index.jsx @@ -7,7 +7,7 @@ import useRestoreConnection from '../../hooks/use-restore-connection/index.jsx'; * Returns a ConnectionErrorNotice component and the conditional flag on whether * to render the component or not. * - * @returns {Object} - The hook data. + * @returns {object} - The hook data. */ export default function useConnectionErrorNotice() { const { connectionErrors } = useConnection( {} ); diff --git a/projects/js-packages/connection/hooks/use-restore-connection/index.jsx b/projects/js-packages/connection/hooks/use-restore-connection/index.jsx index 5ad3ee36b00..87b44ab3168 100644 --- a/projects/js-packages/connection/hooks/use-restore-connection/index.jsx +++ b/projects/js-packages/connection/hooks/use-restore-connection/index.jsx @@ -11,7 +11,7 @@ const { apiRoot, apiNonce } = window?.JP_CONNECTION_INITIAL_STATE * Restore connection hook. * It will initiate an API request attempting to restore the connection, or reconnect if it cannot be restored. * - * @returns {Object} - The hook data. + * @returns {object} - The hook data. */ export default function useRestoreConnection() { const [ isRestoringConnection, setIsRestoringConnection ] = useState( false ); @@ -25,7 +25,7 @@ export default function useRestoreConnection() { * Initiate connection restore. * * @param {boolean} autoReconnectUser - If user connection needs to be reestablished, automatically initiate the flow. - * @returns {Promise} - The API request promise. + * @returns {Promise} - The API request promise. */ const restoreConnection = ( autoReconnectUser = true ) => { setIsRestoringConnection( true ); diff --git a/projects/js-packages/connection/package.json b/projects/js-packages/connection/package.json index 63fea427335..ae4236bd40d 100644 --- a/projects/js-packages/connection/package.json +++ b/projects/js-packages/connection/package.json @@ -1,6 +1,6 @@ { "name": "@automattic/jetpack-connection", - "version": "0.24.5", + "version": "0.24.6-alpha", "description": "Jetpack Connection Component", "author": "Automattic", "license": "GPL-2.0-or-later", diff --git a/projects/js-packages/eslint-changed/changelog/renovate-major-eslint-packages b/projects/js-packages/eslint-changed/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/js-packages/eslint-changed/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/js-packages/eslint-changed/tests/cli.test.js b/projects/js-packages/eslint-changed/tests/cli.test.js index 666e80a4433..cd93c77c3de 100644 --- a/projects/js-packages/eslint-changed/tests/cli.test.js +++ b/projects/js-packages/eslint-changed/tests/cli.test.js @@ -305,9 +305,9 @@ describe( 'bin/eslint-changed.js', () => { * @param {object[]} branches - An array of branches to create. * @param {string} [branches.name] - Name of the branch. * @param {string} [branches.parent] - Name of the parent branch. If omitted, the parent is the previous entry in the array. Must be omitted in the first entry. - * @param {object} branches.files - Files to modify, and their contents (or null to delete the file). - * @param {object} [staged] - Files to modify and stage. - * @param {object} [unstaged] - Files to modify and leave unstaged. + * @param {Object} branches.files - Files to modify, and their contents (or null to delete the file). + * @param {Object} [staged] - Files to modify and stage. + * @param {Object} [unstaged] - Files to modify and leave unstaged. */ async function mktmpdirgit( branches, staged, unstaged ) { await mktmpdir(); @@ -326,7 +326,7 @@ describe( 'bin/eslint-changed.js', () => { /** * Modify files. * - * @param {object} files - Files to modify, and their contents (or null to delete the file). + * @param {Object} files - Files to modify, and their contents (or null to delete the file). * @param {boolean} git - Whether to do git manipulations. */ async function doFiles( files, git ) { diff --git a/projects/js-packages/i18n-check-webpack-plugin/changelog/renovate-major-eslint-packages b/projects/js-packages/i18n-check-webpack-plugin/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/js-packages/i18n-check-webpack-plugin/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/js-packages/i18n-check-webpack-plugin/package.json b/projects/js-packages/i18n-check-webpack-plugin/package.json index ea330b92e8c..34811aff04f 100644 --- a/projects/js-packages/i18n-check-webpack-plugin/package.json +++ b/projects/js-packages/i18n-check-webpack-plugin/package.json @@ -1,6 +1,6 @@ { "name": "@automattic/i18n-check-webpack-plugin", - "version": "1.0.25", + "version": "1.0.26-alpha", "description": "A Webpack plugin to check that WordPress i18n hasn't been mangled by Webpack optimizations.", "homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/i18n-check-webpack-plugin/#readme", "bugs": { diff --git a/projects/js-packages/i18n-check-webpack-plugin/src/GettextExtractor.js b/projects/js-packages/i18n-check-webpack-plugin/src/GettextExtractor.js index 15cc8a40033..814061fbf21 100644 --- a/projects/js-packages/i18n-check-webpack-plugin/src/GettextExtractor.js +++ b/projects/js-packages/i18n-check-webpack-plugin/src/GettextExtractor.js @@ -40,7 +40,7 @@ class GettextExtractor { * * @param {object} options - Configuration options. * @param {object} options.babelOptions - Options for Babel. - * @param {object} options.functions - Functions to extract. Defaults are available as a static property `defaultFunctions`. + * @param {Object} options.functions - Functions to extract. Defaults are available as a static property `defaultFunctions`. * @param {Function} options.lintLogger - Lint logging callback. See `this.setLintLogger()`. */ constructor( options = {} ) { diff --git a/projects/js-packages/image-guide/changelog/renovate-major-eslint-packages b/projects/js-packages/image-guide/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/js-packages/image-guide/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/js-packages/image-guide/package.json b/projects/js-packages/image-guide/package.json index 9304288201c..e98431cb70d 100644 --- a/projects/js-packages/image-guide/package.json +++ b/projects/js-packages/image-guide/package.json @@ -1,6 +1,6 @@ { "name": "@automattic/jetpack-image-guide", - "version": "0.1.2", + "version": "0.1.3-alpha", "description": "Go through the dom to analyze image size on screen vs actual file size.", "homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/image-guide/#readme", "type": "module", diff --git a/projects/js-packages/image-guide/src/MeasurableImage.ts b/projects/js-packages/image-guide/src/MeasurableImage.ts index 3f2350dd9ed..35f98b26bca 100644 --- a/projects/js-packages/image-guide/src/MeasurableImage.ts +++ b/projects/js-packages/image-guide/src/MeasurableImage.ts @@ -61,7 +61,7 @@ export class MeasurableImage { * the image size on page has to be multiplied by the device pixel ratio. * * @param {Dimensions} sizeOnPage - The size of the image on the page. - * @returns {Object} - The expected size of the image. + * @returns {object} - The expected size of the image. */ public getExpectedSize( sizeOnPage: Dimensions ) { const dpr = window.devicePixelRatio || 1; diff --git a/projects/js-packages/image-guide/src/find-image-elements.ts b/projects/js-packages/image-guide/src/find-image-elements.ts index efc5c27d9ac..4e77e4f10cb 100644 --- a/projects/js-packages/image-guide/src/find-image-elements.ts +++ b/projects/js-packages/image-guide/src/find-image-elements.ts @@ -48,6 +48,7 @@ export function backgroundImageSource( node: HTMLElement ) { if ( url && url[ 1 ] && imageLikeURL( url[ 1 ] ) ) { return url[ 1 ]; } + return null; } /** diff --git a/projects/js-packages/publicize-components/changelog/renovate-major-eslint-packages b/projects/js-packages/publicize-components/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/js-packages/publicize-components/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/js-packages/publicize-components/package.json b/projects/js-packages/publicize-components/package.json index 00f60652d79..15cef0b069a 100644 --- a/projects/js-packages/publicize-components/package.json +++ b/projects/js-packages/publicize-components/package.json @@ -1,7 +1,7 @@ { "private": true, "name": "@automattic/jetpack-publicize-components", - "version": "0.13.0", + "version": "0.13.1-alpha", "description": "A library of JS components required by the Publicize editor plugin", "homepage": "https://github.com/Automattic/jetpack/tree/HEAD/projects/js-packages/publicize-components/#readme", "bugs": { diff --git a/projects/js-packages/publicize-components/src/components/media-section/index.js b/projects/js-packages/publicize-components/src/components/media-section/index.js index be43784bb41..968b8753cf2 100644 --- a/projects/js-packages/publicize-components/src/components/media-section/index.js +++ b/projects/js-packages/publicize-components/src/components/media-section/index.js @@ -18,7 +18,7 @@ import styles from './styles.module.scss'; /** * Get relevant details from a WordPress media object. * - * @param {Object} media - WordPress media object. + * @param {object} media - WordPress media object. * @returns {{ * mediaData: {width: number, height: number, sourceUrl: string}, * metaData: {mime: string, fileSize: number, length: number} diff --git a/projects/js-packages/publicize-components/src/hooks/use-attached-media/index.js b/projects/js-packages/publicize-components/src/hooks/use-attached-media/index.js index 7bcd0d9f61b..336cf6cadc8 100644 --- a/projects/js-packages/publicize-components/src/hooks/use-attached-media/index.js +++ b/projects/js-packages/publicize-components/src/hooks/use-attached-media/index.js @@ -3,7 +3,7 @@ import { store as editorStore } from '@wordpress/editor'; import { useCallback } from '@wordpress/element'; /** - * @typedef {Object} AttachedMediaHook + * @typedef {object} AttachedMediaHook * @property {Array} attachedMedia - List of media with ID, URL, and metadata. * @property {Function} updateAttachedMedia - Callback used to update the attached media. */ diff --git a/projects/js-packages/publicize-components/src/hooks/use-social-media-message/index.js b/projects/js-packages/publicize-components/src/hooks/use-social-media-message/index.js index 710de65849f..2c660ccd99e 100644 --- a/projects/js-packages/publicize-components/src/hooks/use-social-media-message/index.js +++ b/projects/js-packages/publicize-components/src/hooks/use-social-media-message/index.js @@ -2,7 +2,7 @@ import { useSelect, useDispatch } from '@wordpress/data'; import { store as editorStore } from '@wordpress/editor'; /** - * @typedef {Object} MessageHook + * @typedef {object} MessageHook * @property {string} message - The text of the message. * @property {number} maxLength - The maximum length of the message. * @property {Function} updateMessage - Callback used to update the message. diff --git a/projects/js-packages/publicize-components/src/store/controls.js b/projects/js-packages/publicize-components/src/store/controls.js index 5e100cbd1ad..3c9d451ee25 100644 --- a/projects/js-packages/publicize-components/src/store/controls.js +++ b/projects/js-packages/publicize-components/src/store/controls.js @@ -3,8 +3,8 @@ import apiFetch from '@wordpress/api-fetch'; /** * Trigger an API Fetch request. * - * @param {Object} action - Action Object. - * @param {Object} action.path - Action path. + * @param {object} action - Action Object. + * @param {object} action.path - Action path. * @returns {Promise} Fetch request promise. */ const fetchFromApi = ( { path } ) => { diff --git a/projects/js-packages/publicize-components/src/store/middlewares.js b/projects/js-packages/publicize-components/src/store/middlewares.js index f9a94a9a18a..3ea96785288 100644 --- a/projects/js-packages/publicize-components/src/store/middlewares.js +++ b/projects/js-packages/publicize-components/src/store/middlewares.js @@ -5,8 +5,8 @@ import effects from './effects'; /** * Applies the custom middlewares used specifically in the Publicize extension. * - * @param {Object} store - Store Object. - * @returns {Object} Update Store Object. + * @param {object} store - Store Object. + * @returns {object} Update Store Object. */ export default function applyMiddlewares( store ) { const middlewares = [ refx( effects ) ]; diff --git a/projects/js-packages/publicize-components/src/store/selectors.js b/projects/js-packages/publicize-components/src/store/selectors.js index 1621c9d7f2f..2b8fa90cbd2 100644 --- a/projects/js-packages/publicize-components/src/store/selectors.js +++ b/projects/js-packages/publicize-components/src/store/selectors.js @@ -98,12 +98,12 @@ export function getTweetStorm( state ) { * Constructs the first tweet to use in the thread. * * @param {object} state - State object. - * @returns {object} The tweet. + * @returns {object|undefined} The tweet. */ export function getFirstTweet( state ) { // This isn't defined properly in the test environment, so we have to skip this function. if ( ! select( 'core' ) ) { - return; + return undefined; } const tweetTemplate = getTweetTemplate( state ); @@ -139,12 +139,12 @@ export function getFirstTweet( state ) { * Constructs the last tweet to use in the thread. * * @param {object} state - State object. - * @returns {object} The tweet. + * @returns {object|undefined} The tweet. */ export function getLastTweet( state ) { // This isn't defined properly in the test environment, so we have to skip this function. if ( ! select( 'core/editor' ) ) { - return; + return undefined; } const { getEditedPostAttribute } = select( 'core/editor' ); @@ -543,7 +543,7 @@ export function getFeatureEnableState() { /** * Get all Jetpack Social options. * - * @returns {Object} Object with Jetpack Social options. + * @returns {object} Object with Jetpack Social options. */ export function getJetpackSocialOptions() { const { getEditedPostAttribute } = select( editorStore ); @@ -554,7 +554,7 @@ export function getJetpackSocialOptions() { /** * Get whether the post has already been shared. * - * @returns {Object} Object with Jetpack Social options. + * @returns {object} Object with Jetpack Social options. */ export function getJetpackSocialPostAlreadyShared() { const { getEditedPostAttribute } = select( editorStore ); diff --git a/projects/packages/backup/changelog/renovate-major-eslint-packages b/projects/packages/backup/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/packages/backup/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/packages/backup/src/class-package-version.php b/projects/packages/backup/src/class-package-version.php index 64cc77a1fa3..f0286c24924 100644 --- a/projects/packages/backup/src/class-package-version.php +++ b/projects/packages/backup/src/class-package-version.php @@ -12,7 +12,7 @@ */ class Package_Version { - const PACKAGE_VERSION = '1.10.7'; + const PACKAGE_VERSION = '1.10.8-alpha'; const PACKAGE_SLUG = 'backup'; diff --git a/projects/packages/backup/src/js/hooks/useCapabilities.js b/projects/packages/backup/src/js/hooks/useCapabilities.js index a2f9be184cf..797b3bfcf29 100644 --- a/projects/packages/backup/src/js/hooks/useCapabilities.js +++ b/projects/packages/backup/src/js/hooks/useCapabilities.js @@ -5,7 +5,7 @@ import useConnection from './useConnection'; /** * Return information and loader of Backup functioality Capabilities * - * @returns {Object} capabilities, capabilitiesError, capabilitiesLoaded, fetchCapabilities + * @returns {object} capabilities, capabilitiesError, capabilitiesLoaded, fetchCapabilities */ export default function useCapabilities() { const [ capabilities, setCapabilities ] = useState( null ); diff --git a/projects/packages/search/changelog/renovate-major-eslint-packages b/projects/packages/search/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/packages/search/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/packages/search/src/dashboard/hooks/use-connection.jsx b/projects/packages/search/src/dashboard/hooks/use-connection.jsx index 7364eae7f1a..8bd1cedd860 100644 --- a/projects/packages/search/src/dashboard/hooks/use-connection.jsx +++ b/projects/packages/search/src/dashboard/hooks/use-connection.jsx @@ -5,7 +5,7 @@ import { STORE_ID as SEARCH_STORE_ID } from 'store'; /** * Expose the `connectionStatus`, `isFullyConnected` state object * - * @returns {Object} connectionStatus, isFullyConnected + * @returns {object} connectionStatus, isFullyConnected */ export default function useConnection() { const connectionStatus = useSelect( select => ( { diff --git a/projects/packages/search/src/instant-search/lib/filters.js b/projects/packages/search/src/instant-search/lib/filters.js index 85a7df9be2e..cf50b9a083c 100644 --- a/projects/packages/search/src/instant-search/lib/filters.js +++ b/projects/packages/search/src/instant-search/lib/filters.js @@ -181,7 +181,7 @@ export function mapFilterKeyToFilter( filterKey ) { * Returns the type of the inputted filter object. * * @param {object} filter - filter key string to be mapped. - * @returns {string} output + * @returns {string|undefined} output */ export function mapFilterToType( filter ) { if ( filter.type === 'date_histogram' ) { @@ -197,4 +197,5 @@ export function mapFilterToType( filter ) { } else if ( filter.type === 'group' ) { return 'group'; } + return undefined; } diff --git a/projects/packages/search/tools/webpack.instant.config.js b/projects/packages/search/tools/webpack.instant.config.js index eca417109f6..aebfcf1f613 100644 --- a/projects/packages/search/tools/webpack.instant.config.js +++ b/projects/packages/search/tools/webpack.instant.config.js @@ -19,7 +19,7 @@ const webpack = jetpackWebpackConfig.webpack; function requestToExternal( request ) { // Ensure that React will be aliased to preact/compat by preventing externalization. if ( request === 'react' || request === 'react-dom' ) { - return; + return undefined; } return defaultRequestToExternal( request ); } diff --git a/projects/plugins/boost/app/assets/src/js/api/speed-scores.ts b/projects/plugins/boost/app/assets/src/js/api/speed-scores.ts index 8177e62e1f4..f2bbc2718e8 100644 --- a/projects/plugins/boost/app/assets/src/js/api/speed-scores.ts +++ b/projects/plugins/boost/app/assets/src/js/api/speed-scores.ts @@ -184,7 +184,7 @@ export function didScoresChange( scores: SpeedScoresSet ): boolean { /** * Determine the change in scores to pass through to other functions. * - * @param scores + * @param scores * @return percentage */ export function getScoreMovementPercentage( scores: SpeedScoresSet ): number { diff --git a/projects/plugins/boost/app/features/image-guide/src/analytics.ts b/projects/plugins/boost/app/features/image-guide/src/analytics.ts index 391376e0b9c..8524e910210 100644 --- a/projects/plugins/boost/app/features/image-guide/src/analytics.ts +++ b/projects/plugins/boost/app/features/image-guide/src/analytics.ts @@ -70,7 +70,7 @@ export default class ImageGuideAnalytics { /** * Track events to record the outcome of the image guide for the current page. * - * @param imageStores + * @param imageStores */ public static async trackPage( imageStores: MeasurableImageStore[] ) { if ( ! imageStores.length || ImageGuideAnalytics.trackingComplete ) { diff --git a/projects/plugins/boost/app/features/image-guide/src/index.ts b/projects/plugins/boost/app/features/image-guide/src/index.ts index 2da8562ad91..108d9e8c37e 100644 --- a/projects/plugins/boost/app/features/image-guide/src/index.ts +++ b/projects/plugins/boost/app/features/image-guide/src/index.ts @@ -11,7 +11,7 @@ import type { MeasurableImage } from '@automattic/jetpack-image-guide'; * that are too small, for example * avatars, icons, etc. * - * @param images An array of images to filter + * @param images An array of images to filter */ function discardSmallImages( images: MeasurableImage[] ) { const minSize = 65; diff --git a/projects/plugins/boost/app/features/image-guide/src/initialize.ts b/projects/plugins/boost/app/features/image-guide/src/initialize.ts index 8f765ef95c3..57fd1906d44 100644 --- a/projects/plugins/boost/app/features/image-guide/src/initialize.ts +++ b/projects/plugins/boost/app/features/image-guide/src/initialize.ts @@ -10,7 +10,7 @@ import type { MeasurableImage } from '@automattic/jetpack-image-guide'; * within the DOM tree, and to prevent it from being obscured * by other elements with a higher z-index. * - * @param node The node to start searching from + * @param node The node to start searching from * @return The closest parent element that is able to contain the image guide component */ function getClosestContainingAncestor( node: HTMLElement ): HTMLElement | null { @@ -117,7 +117,7 @@ function findContainer( image: MeasurableImage ): HTMLElement | undefined { * * This function attempts to attach the Svelte Components to the DOM in a non-destructive way. * - * @param measuredImages + * @param measuredImages */ export function attachGuides( measuredImages: MeasurableImage[] ) { const componentConfiguration = measuredImages.reduce( ( acc, image ) => { diff --git a/projects/plugins/boost/changelog/renovate-major-eslint-packages b/projects/plugins/boost/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..9af19ad0939 --- /dev/null +++ b/projects/plugins/boost/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: fixed + +Clean up JavaScript eslint issues. diff --git a/projects/plugins/boost/composer.json b/projects/plugins/boost/composer.json index 4c50c765592..d7902407ecd 100644 --- a/projects/plugins/boost/composer.json +++ b/projects/plugins/boost/composer.json @@ -3,7 +3,7 @@ "description": "Boost your WordPress site's performance, from the creators of Jetpack", "type": "library", "license": "GPL-2.0-or-later", - "version": "1.7.0", + "version": "1.7.1-alpha", "authors": [ { "name": "Automattic, Inc.", @@ -71,7 +71,7 @@ "platform": { "ext-intl": "0.0.0" }, - "autoloader-suffix": "b1e77e6231d50e7663f84529b6a3dfda_jetpack_boostâ“¥1_7_0", + "autoloader-suffix": "b1e77e6231d50e7663f84529b6a3dfda_jetpack_boostâ“¥1_7_1_alpha", "allow-plugins": { "roots/wordpress-core-installer": true, "automattic/jetpack-autoloader": true, diff --git a/projects/plugins/boost/composer.lock b/projects/plugins/boost/composer.lock index 69b6ebf2898..b694318e613 100644 --- a/projects/plugins/boost/composer.lock +++ b/projects/plugins/boost/composer.lock @@ -4,7 +4,7 @@ "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "This file is @generated automatically" ], - "content-hash": "6d8f939e6806f63483480f8b82ba2583", + "content-hash": "f4bb747a13beccc4926a371351e628a6", "packages": [ { "name": "automattic/jetpack-a8c-mc-stats", diff --git a/projects/plugins/boost/jetpack-boost.php b/projects/plugins/boost/jetpack-boost.php index 300430707ab..09775cc72bf 100644 --- a/projects/plugins/boost/jetpack-boost.php +++ b/projects/plugins/boost/jetpack-boost.php @@ -9,7 +9,7 @@ * Plugin Name: Jetpack Boost * Plugin URI: https://jetpack.com/boost * Description: Boost your WordPress site's performance, from the creators of Jetpack - * Version: 1.7.0 + * Version: 1.7.1-alpha * Author: Automattic - Jetpack Site Speed team * Author URI: https://jetpack.com/boost/ * License: GPL-2.0+ @@ -29,7 +29,7 @@ die; } -define( 'JETPACK_BOOST_VERSION', '1.7.0' ); +define( 'JETPACK_BOOST_VERSION', '1.7.1-alpha' ); define( 'JETPACK_BOOST_SLUG', 'jetpack-boost' ); if ( ! defined( 'JETPACK_BOOST_CLIENT_NAME' ) ) { diff --git a/projects/plugins/boost/package.json b/projects/plugins/boost/package.json index 4a131334f0e..cf47dcd4a63 100644 --- a/projects/plugins/boost/package.json +++ b/projects/plugins/boost/package.json @@ -1,6 +1,6 @@ { "name": "jetpack-boost", - "version": "1.7.0", + "version": "1.7.1-alpha", "description": "Boost your WordPress site's performance, from the creators of Jetpack", "directories": { "test": "tests" diff --git a/projects/plugins/jetpack/_inc/client/recommendations/utils/compute-max-suggested-discount.js b/projects/plugins/jetpack/_inc/client/recommendations/utils/compute-max-suggested-discount.js index edef57e2caa..f17be755535 100644 --- a/projects/plugins/jetpack/_inc/client/recommendations/utils/compute-max-suggested-discount.js +++ b/projects/plugins/jetpack/_inc/client/recommendations/utils/compute-max-suggested-discount.js @@ -1,20 +1,20 @@ /** * Return the highest discount amongst all suggested products. * - * @param {Object} discountData - Informations about the discount - * @param {Object[]} introOffers - Set of product offering information - * @param {Object[]} suggestions - Suggested products - * @returns {number} Max discount + * @param {object} discountData - Informations about the discount + * @param {object[]} introOffers - Set of product offering information + * @param {object[]} suggestions - Suggested products + * @returns {number|undefined} Max discount */ export const computeMaxSuggestedDiscount = ( discountData, introOffers, suggestions ) => { if ( ! discountData || ! introOffers || ! suggestions ) { - return; + return undefined; } const discount = parseInt( discountData?.discount, 10 ); if ( Number.isNaN( discount ) ) { - return; + return undefined; } const slugs = suggestions.map( ( { slug } ) => slug ); @@ -22,7 +22,7 @@ export const computeMaxSuggestedDiscount = ( discountData, introOffers, suggesti const discounts = offers .map( ( { original_price: originalPrice, raw_price: introPrice } ) => { if ( ! originalPrice ) { - return; + return undefined; } const finalPrice = introPrice * ( 1 - discount / 100 ); @@ -33,7 +33,7 @@ export const computeMaxSuggestedDiscount = ( discountData, introOffers, suggesti .filter( Boolean ); if ( ! discounts.length ) { - return; + return undefined; } return Math.max( ...discounts ); diff --git a/projects/plugins/jetpack/changelog/renovate-major-eslint-packages b/projects/plugins/jetpack/changelog/renovate-major-eslint-packages new file mode 100644 index 00000000000..ea1fd9f0ff4 --- /dev/null +++ b/projects/plugins/jetpack/changelog/renovate-major-eslint-packages @@ -0,0 +1,4 @@ +Significance: patch +Type: other + +Clean up JavaScript eslint issues. diff --git a/projects/plugins/jetpack/extensions/blocks/dialogue/components/participants-control.js b/projects/plugins/jetpack/extensions/blocks/dialogue/components/participants-control.js index 68a6d605496..64efb948751 100644 --- a/projects/plugins/jetpack/extensions/blocks/dialogue/components/participants-control.js +++ b/projects/plugins/jetpack/extensions/blocks/dialogue/components/participants-control.js @@ -216,6 +216,8 @@ export function SpeakerEditControl( { } else { setEditingMode( EDIT_MODE_ADDING ); } + + return null; } // Keep autocomplete options udated. diff --git a/projects/plugins/jetpack/extensions/blocks/google-calendar/utils.js b/projects/plugins/jetpack/extensions/blocks/google-calendar/utils.js index e66e621cf31..7b94f92e71c 100644 --- a/projects/plugins/jetpack/extensions/blocks/google-calendar/utils.js +++ b/projects/plugins/jetpack/extensions/blocks/google-calendar/utils.js @@ -18,12 +18,12 @@ const ATTRIBUTE_REGEX = /\s+(\w+)=(["'])(.*?)\2/gi; * to an embed URL. * * @param {string} shareableUrl - The Google Calendar shareable URL - * @returns {string} The embed URL or undefined if the conversion fails + * @returns {string|undefined} The embed URL or undefined if the conversion fails */ export function convertShareableUrl( shareableUrl ) { const parsedUrl = SHAREABLE_REGEX.exec( shareableUrl ); if ( ! parsedUrl ) { - return; + return undefined; } return ( 'https://calendar.google.com/calendar/embed?src=' + encodeURIComponent( atob( parsedUrl[ 1 ] ) ) @@ -34,25 +34,25 @@ export function convertShareableUrl( shareableUrl ) { * Given an