alovajs / alova

Workflow-Streamlined next-generation request tools. Extremely streamline API integration workflow, just one step
https://alova.js.org
MIT License
2.97k stars 109 forks source link

feat(solid): add solid statesHook #479

Closed theqianqiu closed 3 weeks ago

theqianqiu commented 4 months ago

相关 Issue / Related Issue

[Issue ID]

这个 PR 是什么类型?/ What type of PR is this?

这个 PR 做了什么?/ What does this PR do?

[简要描述所做更改 / Describe the changes briefly]

文档 / Docs

[此次 PR 的文档 / Docs for this PR]

测试 / Testing

[描述测试结果 / Describe the test results.]

changeset-bot[bot] commented 4 months ago

🦋 Changeset detected

Latest commit: be7bc7fbab4134f0e1172a476c46dd0349ae364d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ----- | ----- | | alova | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

MeetinaXD commented 3 months ago

感谢 PR!目前发现了以下问题:

  1. 过半的更改都是关于修复 'React' must be in scope when using JSX 这个问题,请考虑在其他 PR 解决,或修改 PR 描述;
  2. 出现了无意义提交,只影响代码风格而不产生实际更改的修改不应该提交;
  3. 修改了根 tsconfig.json,具体请看文件中的描述;
  4. 新增的测试没通过

稍后记得补充 changeset 😃