Skip to content

Commit e6fcf3b

Browse files
authored
revert(eslint-plugin): delete custom rules (#81)
* doc(eslint-plugin): no more custom rules * fix(eslint-plugin): delete example of custom rules * BREAKING CHANGE(eslint-plugin): delete custom rules * doc(eslint-plugin): delete custom rules section * fix(eslint-plugin): remove auto-generating doc command * chore(eslint-plugin): delete CI test check
1 parent 29106d5 commit e6fcf3b

27 files changed

Lines changed: 1 addition & 1141 deletions

.github/workflows/quality-eslint.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,3 @@ jobs:
3838

3939
- name: Linter
4040
run: yarn lint
41-
42-
- name: Tests
43-
run: yarn test

example-app/eslint-breaking-examples/break-accessibility-props-require-accessible.jsx

Lines changed: 0 additions & 16 deletions
This file was deleted.

example-app/eslint-breaking-examples/break-accessible-requires-accessibilityLabel.tsx

Lines changed: 0 additions & 20 deletions
This file was deleted.

example-app/eslint-breaking-examples/break-accessible-requires-accessibilityRole.tsx

Lines changed: 0 additions & 18 deletions
This file was deleted.

example-app/eslint-breaking-examples/break-do-not-use-role-on-image.tsx

Lines changed: 0 additions & 26 deletions
This file was deleted.

example-app/eslint-breaking-examples/break-image-requires-accessible-prop.tsx

Lines changed: 0 additions & 20 deletions
This file was deleted.

example-app/eslint-breaking-examples/break-require-named-effect.tsx

Lines changed: 0 additions & 19 deletions
This file was deleted.

packages/eslint-plugin/README.md

Lines changed: 0 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -94,40 +94,6 @@ If you use **npm >= 5** you can automatically install them by running:
9494
npx install-peerdeps @bam.tech/eslint-config -D
9595
```
9696

97-
## Eslint rules
98-
99-
This plugin exports some custom rules that you can optionally use in your project:
100-
101-
<!-- begin auto-generated rules list -->
102-
103-
🔧 Automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/user-guide/command-line-interface#--fix).\
104-
💡 Manually fixable by [editor suggestions](https://eslint.org/docs/developer-guide/working-with-rules#providing-suggestions).
105-
106-
| Name                                        | Description | 🔧 | 💡 |
107-
| :---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :-------------------------------------------------------------- | :-- | :-- |
108-
| [accessibility-props-require-accessible](https://github.com/bamlab/react-native-project-config/blob/main/packages/eslint-plugin/docs/rules/accessibility-props-require-accessible.md) | Requires accessible prop when accessibility props are defined | 🔧 | |
109-
| [do-not-use-role-on-image](https://github.com/bamlab/react-native-project-config/blob/main/packages/eslint-plugin/docs/rules/do-not-use-role-on-image.md) | Disallow role prop on Image component | 🔧 | |
110-
| [image-requires-accessible-prop](https://github.com/bamlab/react-native-project-config/blob/main/packages/eslint-plugin/docs/rules/image-requires-accessible-prop.md) | Require accessible prop on image components | 🔧 | 💡 |
111-
| [require-named-effect](https://github.com/bamlab/react-native-project-config/blob/main/packages/eslint-plugin/docs/rules/require-named-effect.md) | Enforces the use of named functions inside a useEffect | | |
112-
| [requires-accessibility-label](https://github.com/bamlab/react-native-project-config/blob/main/packages/eslint-plugin/docs/rules/requires-accessibility-label.md) | Enforces label when component accessible | | 💡 |
113-
| [requires-accessibility-role-when-accessible](https://github.com/bamlab/react-native-project-config/blob/main/packages/eslint-plugin/docs/rules/requires-accessibility-role-when-accessible.md) | Enforces accessibilityRole or role when component is accessible | | 💡 |
114-
115-
<!-- end auto-generated rules list -->
116-
117-
To use a rule, just declare it in your `.eslintrc`:
118-
119-
```json
120-
// .eslintrc
121-
{
122-
"plugins": ["@bam.tech"],
123-
"rules": {
124-
"@bam.tech/require-named-effect": "error"
125-
}
126-
}
127-
```
128-
129-
> _Tip: if your config is already extended from a `@bam.tech` config, you don't need to declare the plugin._
130-
13197
## How to customize?
13298

13399
You can still customize your ESLint config by adding other configurations, plugins and rules to your `.eslintrc` config file.

packages/eslint-plugin/docs/rules/accessibility-props-require-accessible.md

Lines changed: 0 additions & 33 deletions
This file was deleted.

packages/eslint-plugin/docs/rules/do-not-use-role-on-image.md

Lines changed: 0 additions & 21 deletions
This file was deleted.

0 commit comments

Comments
 (0)