Skip to content
This repository was archived by the owner on Mar 7, 2025. It is now read-only.

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.

@azat-io azat-io requested a review from Shinigami92 as a code owner April 23, 2023 20:37
@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
Member

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
Member

@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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants