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

docs: delete stream.md && supplement stream description to client #752

Merged
merged 8 commits into from
Sep 7, 2023

Conversation

chaoranz758
Copy link
Member

@chaoranz758 chaoranz758 commented Aug 5, 2023

What type of PR is this?

docs

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.

删除独立的流式处理文件,并在client和engine中补充说明流式处理的内容

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:docs: delete stream.md && supplement stream description to client
zh(optional): docs: 删除独立的流式处理文件,并在client和engine中补充说明流式处理的内容

(Optional) Which issue(s) this PR fixes:

Fixes #606

@chaoranz758 chaoranz758 requested review from a team as code owners August 5, 2023 08:02
@vercel
Copy link

vercel bot commented Aug 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cloudwego ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 20, 2023 5:28am

@li-jin-gou
Copy link
Member

@chaoranz758 确定一下流式的文档,还有别的地方引用嘛

@rogerogers
Copy link
Contributor

更多详细的设计说明可以参考:[Hertz 支持 QUIC & HTTP/3](https://www.cloudwego.io/zh/docs/hertz/reference/stream_based_design/)

这个链接不存在,这个stream是这里吗

@li-jin-gou
Copy link
Member

更多详细的设计说明可以参考:[Hertz 支持 QUIC & HTTP/3](https://www.cloudwego.io/zh/docs/hertz/reference/stream_based_design/)

这个链接不存在,这个stream是这里吗

这个 pr 引入的问题 4d259fc cc @chaoranz758

@li-jin-gou li-jin-gou merged commit 622a693 into cloudwego:main Sep 7, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Optimise(docs): Improving Hertz's stream documentation
4 participants