Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for React #193

Merged
merged 6 commits into from
Apr 25, 2023
Merged

Add support for React #193

merged 6 commits into from
Apr 25, 2023

Conversation

azat-io
Copy link
Contributor

@azat-io azat-io commented Apr 23, 2023

https://github.com/jsx-eslint/eslint-plugin-react

Generated with pnpm run generate:rules.

Also tried to generate typings for eslint-plugin-react-hooks, but there are some problems now.

package.json Outdated Show resolved Hide resolved
tsconfig.json Outdated Show resolved Hide resolved
@Dimava
Copy link
Contributor

Dimava commented Apr 24, 2023

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Source Errors ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

Vitest found 1 error not related to your test files.
TypeCheckError: Type 'JsxNoConstructedContextValuesOption' does not satisfy the constraint 'any[]'.
  Type 'JsxNoConstructedContextValuesOption' is missing the following properties from type 'any[]' length, pop, push, concat, and 29 more.
  src/rules/react/jsx-no-constructed-context-values.d.ts:22:14
     20|  */
     21| export type JsxNoConstructedContextValuesRuleConfig =
     22|   RuleConfig<JsxNoConstructedContextValuesOptions>;
       |              ^
     23| 
     24| /**

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  Tests failed. Watching for file changes...

Yet another root-schema-{}-not-[] problem

Please cherry-pick e43693a

@Shinigami92 Shinigami92 marked this pull request as draft April 25, 2023 16:44
@Shinigami92 Shinigami92 added the enhancement New feature or request label Apr 25, 2023
@Shinigami92 Shinigami92 marked this pull request as ready for review April 25, 2023 16:52
@Shinigami92
Copy link
Collaborator

Shinigami92 commented Apr 25, 2023

@azat-io please check my latest commits and learn from that, so you can produce settings and extends in future PRs

@Shinigami92 Shinigami92 merged commit 5e0578a into eslint-types:main Apr 25, 2023
@Shinigami92
Copy link
Collaborator

@azat-io I will wait for your PR for https://www.npmjs.com/package/eslint-plugin-react-hooks before marking next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants