-
Notifications
You must be signed in to change notification settings - Fork 58
添加单元测试,基于vitest 改造了现有的测试。 #385
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
Conversation
🦋 Changeset detectedLatest commit: c4a421a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
rookie-luochao
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
感谢大佬贡献pr, 我先看看
|
合并代码之后,有些pipeline跑不过了呢 https://github.com/openapi-ui/openapi-ts-request/actions/runs/16070733161 |
|
在我这个仓库,好像跑pipeline执行单测过不了 |
我看看原因,起码说明pipeline配置对了,每次用unit 拦截的效果也有了 |
|
是的,拦截是有效果的 |
|
看起来新的另外这个PR中的代码有问题, 我看一下, 等下再来一个PR |
|
感觉那个pr和这个没太多关联?感谢大佬定位
发自我的iPhone
…------------------ 原始邮件 ------------------
发件人: Kevin Law ***@***.***>
发送时间: 2025年7月4日 18:19
收件人: openapi-ui/openapi-ts-request ***@***.***>
抄送: 故城 ***@***.***>, State change ***@***.***>
主题: Re: [openapi-ui/openapi-ts-request] 添加单元测试,基于vitest 改造了现有的测试。 (PR #385)
|
|
提了一个新的PR #390 解决了, 你看下。 |
这个PR的主要修改内容为:
看看还有没有什么遗漏,请告知