diff --git a/.changeset/eighty-tips-deliver.md b/.changeset/eighty-tips-deliver.md new file mode 100644 index 00000000..debe7004 --- /dev/null +++ b/.changeset/eighty-tips-deliver.md @@ -0,0 +1,5 @@ +--- +"eslint-plugin-primer-react": patch +--- + +Update message for no-deprecated-experimental-components rule diff --git a/.changeset/honest-adults-add.md b/.changeset/honest-adults-add.md new file mode 100644 index 00000000..9293092a --- /dev/null +++ b/.changeset/honest-adults-add.md @@ -0,0 +1,5 @@ +--- +"eslint-plugin-primer-react": patch +--- + +Removes primer-react/enforce-css-module-identifier-casing, primer-react/enforce-css-module-default-import from recommended set of rules. diff --git a/package-lock.json b/package-lock.json index b0282a88..164274ac 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,12 +1,12 @@ { "name": "eslint-plugin-primer-react", - "version": "6.1.6", + "version": "7.0.0", "lockfileVersion": 2, "requires": true, "packages": { "": { "name": "eslint-plugin-primer-react", - "version": "6.1.6", + "version": "7.0.0", "license": "MIT", "dependencies": { "@styled-system/props": "^5.1.5", diff --git a/src/configs/recommended.js b/src/configs/recommended.js index 1d4a9f33..61cd07cc 100644 --- a/src/configs/recommended.js +++ b/src/configs/recommended.js @@ -22,8 +22,6 @@ module.exports = { 'primer-react/a11y-use-accessible-tooltip': 'error', 'primer-react/no-unnecessary-components': 'error', 'primer-react/prefer-action-list-item-onselect': 'error', - 'primer-react/enforce-css-module-identifier-casing': 'error', - 'primer-react/enforce-css-module-default-import': ['error', {enforceName: '(^classes$|Classes$)'}], }, settings: { github: { diff --git a/src/rules/__tests__/no-deprecated-experimental-components.test.js b/src/rules/__tests__/no-deprecated-experimental-components.test.js index 3488328a..bcdc0358 100644 --- a/src/rules/__tests__/no-deprecated-experimental-components.test.js +++ b/src/rules/__tests__/no-deprecated-experimental-components.test.js @@ -30,14 +30,14 @@ ruleTester.run('no-deprecated-experimental-components', rule, { { code: `import {SelectPanel} from '@primer/react/experimental'`, errors: [ - 'SelectPanel is deprecated. Please import from the stable entrypoint (@primer/react) if available, or check https://primer.style/product/components/ for alternative components.', + 'The experimental SelectPanel is deprecated. Please import from the stable entrypoint (@primer/react) if available, or check https://primer.style/product/components/ for alternative components.', ], }, // Multiple experimental import { code: `import {SelectPanel, DataTable, ActionBar} from '@primer/react/experimental'`, errors: [ - 'SelectPanel is deprecated. Please import from the stable entrypoint (@primer/react) if available, or check https://primer.style/product/components/ for alternative components.', + 'The experimental SelectPanel is deprecated. Please import from the stable entrypoint (@primer/react) if available, or check https://primer.style/product/components/ for alternative components.', ], }, ], diff --git a/src/rules/no-deprecated-experimental-components.js b/src/rules/no-deprecated-experimental-components.js index 3746c9fa..2abe522d 100644 --- a/src/rules/no-deprecated-experimental-components.js +++ b/src/rules/no-deprecated-experimental-components.js @@ -52,7 +52,8 @@ module.exports = { } if (experimental.length > 0) { - const message = `${components.join(', ')} ${ + // eslint-disable-next-line i18n-text/no-en + const message = `The experimental ${components.join(', ')} ${ components.length > 1 ? 'are' : 'is' } deprecated. Please import from the stable entrypoint (@primer/react) if available, or check https://primer.style/product/components/ for alternative components.`