Merge branch 'jsx-eslint:master' into master

Author: ROSSROSALES

.eslintrc

@@ -67,5 +67,16 @@
           "no-console": 0,
         },
       },
+      {
+        "files": ".github/workflows/*.js",
+        "parserOptions": {
+          "ecmaVersion": 2019,
+        },
+        "rules": {
+          "camelcase": 0,
+          "no-console": 0,
+          "no-restricted-syntax": 0,
+        },
+      },
     ],
 }

.github/workflows/npm-publish.yml

@@ -48,39 +48,8 @@ jobs:
           result-encoding: string
           retries: 3
           script: |
-            const ref = context.payload.inputs.tag;
-
-            console.log(`Checking status checks for ${ref}`);
-
-            const { owner, repo } = context.repo;
-            const { default_branch: branch } = context.payload.repository;
-
-            const branch = github.rest.repos.getBranch({ owner, repo, branch });
-
-            const checkSuites = await github.rest.checks.listSuitesForRef({ owner, repo, ref });
-
-            if (checkSuites.some(({ status }) => 'completed')) {
-              core.setFailed(`Some workflows for ${context.payload.inputs.tag} are still in-progress`);
-            }
-
-            const { data: { check_runs: checkRuns } } = await Promise.all(
-              (await branch).data.protection.required_status_checks.checks.map(({ context }) => (
-                github.rest.checks.listForRef({
-                  owner,
-                  repo,
-                  ref,
-                  check_name: context
-                })
-              )
-            )
-
-            checkRuns.forEach(({ name, status, conclusion }) => {
-              if (status !== 'completed' || conclusion !== 'success') {
-                console.log(`${name} check failed`);
-                core.setFailed(`Required status check ${name} did not succeed`);
-              }
-              console.log(`${name} check passed`);
-            });
+            const script = require('./.github/workflows/publish');
+            console.log(await script({ github, context, core }));
 
   publish:
     needs: [check-status]
@@ -109,7 +78,7 @@ jobs:
         env:
           NPM_CONFIG_LEGACY_PEER_DEPS: true
 
-      - run: echo "//registry.npmjs.org/:_authToken=${{ secrets.NODE_AUTH_TOKEN }}" >> .npmrc
+      - run: echo "//registry.npmjs.org/:_authToken=${{ secrets.NPM_TOKEN }}" >> .npmrc
 
       - run: npm publish --dry-run
 

.github/workflows/publish.js

@@ -0,0 +1,41 @@
+'use strict';
+
+module.exports = async function publish({ github, context, core }) {
+  const ref = context.payload.inputs.tag;
+
+  console.log(`Checking status checks for ${ref}`);
+
+  const { owner, repo } = context.repo;
+  const { default_branch: branch } = context.payload.repository;
+
+  const branchData = github.rest.repos.getBranch({ owner, repo, branch });
+
+  const { data: { check_suites: checkSuites } } = await github.rest.checks.listSuitesForRef({ owner, repo, ref });
+
+  if (checkSuites.some(({ status }) => status === 'completed')) {
+    core.setFailed(`Some workflows for ${context.payload.inputs.tag} are still in-progress`);
+  }
+
+  const result = await Promise.all(
+    (await branchData).data.protection.required_status_checks.checks.map(({ context: check_name }) => (
+      github.rest.checks.listForRef({
+        owner,
+        repo,
+        ref,
+        check_name,
+      })
+    )),
+  );
+
+  console.log(result);
+
+  const { data: { check_runs: checkRuns } } = result;
+
+  checkRuns.forEach(({ name, status, conclusion }) => {
+    if (status !== 'completed' || conclusion !== 'success') {
+      console.log(`${name} check failed`);
+      core.setFailed(`Required status check ${name} did not succeed`);
+    }
+    console.log(`${name} check passed`);
+  });
+};

CHANGELOG.md

@@ -15,12 +15,14 @@ This change log adheres to standards from [Keep a CHANGELOG](https://keepachange
 * configs: avoid legacy config system error ([#3461][] @ljharb)
 * [`jsx-no-target-blank`]: allow ternaries with literals ([#3464][] @akulsr0)
 * [`sort-prop-types`]: restore autofixing ([#2574][] @ROSSROSALES)
+* [`no-unknown-property`]: add `inert` attribute ([#3484][] @ljharb)
 
 ### Changed
 * [Perf] component detection: improve performance by avoiding traversing parents unnecessarily ([#3459][] @golopot)
 * [Docs] `forbid-component-props`: inclusive language w/ allowlist ([#3473][] @AndersDJohnson)
 * [Docs] automate doc generation with `eslint-doc-generator` ([#3469][] @bmish)
 
+[#3484]: https://github.com/jsx-eslint/eslint-plugin-react/issues/3484
 [#3473]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3473
 [#3469]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3469
 [#3464]: https://github.com/jsx-eslint/eslint-plugin-react/pull/3464

lib/rules/no-unknown-property.js

@@ -202,7 +202,7 @@ const SVGDOM_ATTRIBUTE_NAMES = {
 const DOM_PROPERTY_NAMES_ONE_WORD = [
   // Global attributes - can be used on any HTML/DOM element
   // See https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes
-  'dir', 'draggable', 'hidden', 'id', 'lang', 'nonce', 'part', 'slot', 'style', 'title', 'translate',
+  'dir', 'draggable', 'hidden', 'id', 'lang', 'nonce', 'part', 'slot', 'style', 'title', 'translate', 'inert',
   // Element specific attributes
   // See https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes (includes global attributes too)
   // To be considered if these should be added also to ATTRIBUTE_TAGS_MAP

package.json

@@ -25,24 +25,24 @@
   "homepage": "https://github.com/jsx-eslint/eslint-plugin-react",
   "bugs": "https://github.com/jsx-eslint/eslint-plugin-react/issues",
   "dependencies": {
-    "array-includes": "^3.1.5",
-    "array.prototype.flatmap": "^1.3.0",
-    "array.prototype.tosorted": "^1.1.0",
+    "array-includes": "^3.1.6",
+    "array.prototype.flatmap": "^1.3.1",
+    "array.prototype.tosorted": "^1.1.1",
     "doctrine": "^2.1.0",
     "estraverse": "^5.3.0",
     "jsx-ast-utils": "^2.4.1 || ^3.0.0",
     "minimatch": "^3.1.2",
-    "object.entries": "^1.1.5",
-    "object.fromentries": "^2.0.5",
-    "object.hasown": "^1.1.1",
-    "object.values": "^1.1.5",
+    "object.entries": "^1.1.6",
+    "object.fromentries": "^2.0.6",
+    "object.hasown": "^1.1.2",
+    "object.values": "^1.1.6",
     "prop-types": "^15.8.1",
-    "resolve": "^2.0.0-next.3",
+    "resolve": "^2.0.0-next.4",
     "semver": "^6.3.0",
-    "string.prototype.matchall": "^4.0.7"
+    "string.prototype.matchall": "^4.0.8"
   },
   "devDependencies": {
-    "@babel/core": "^7.19.3",
+    "@babel/core": "^7.20.2",
     "@babel/eslint-parser": "^7.19.1",
     "@babel/plugin-syntax-decorators": "^7.19.0",
     "@babel/plugin-syntax-do-expressions": "^7.18.6",

tests/lib/rules/no-unknown-property.js

@@ -51,7 +51,7 @@ ruleTester.run('no-unknown-property', rule, {
     { code: '<input ref={this.input} type="radio" />' },
     { code: '<input type="file" webkitdirectory="" />' },
     { code: '<input type="file" webkitDirectory="" />' },
-    { code: '<div children="anything" />' },
+    { code: '<div inert children="anything" />' },
     { code: '<iframe scrolling="?" onLoad={a} onError={b} align="top" />' },
     { code: '<input key="bar" type="radio" />' },
     { code: '<button disabled>You cannot click me</button>;' },
@@ -62,7 +62,7 @@ ruleTester.run('no-unknown-property', rule, {
     { code: '<input type="checkbox" checked={checked} disabled={disabled} id={id} onChange={onChange} />' },
     { code: '<video playsInline />' },
     { code: '<img onError={foo} onLoad={bar} />' },
-    { code: '<picture onError={foo} onLoad={bar} />' },
+    { code: '<picture inert={false} onError={foo} onLoad={bar} />' },
     { code: '<iframe onError={foo} onLoad={bar} />' },
     { code: '<script onLoad={bar} onError={foo} />' },
     { code: '<source onError={foo} />' },