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

feat(hz): use third model as "import" of handlers #619

Merged
merged 1 commit into from
Feb 20, 2023

Conversation

FGYFFFF
Copy link
Contributor

@FGYFFFF FGYFFFF commented Feb 17, 2023

What type of PR is this?

feat

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.

使用第三方 model 作为生成代码中 handler 的依赖

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

en: use third model as "import" of handlers
zh(optional): 使用第三方 model 作为生成代码中 handler 的依赖

Which issue(s) this PR fixes:

@FGYFFFF FGYFFFF requested review from a team as code owners February 17, 2023 07:19
@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Base: 72.19% // Head: 72.19% // No change to project coverage 👍

Coverage data is based on head (afd118f) compared to base (6f37d9d).
Patch has no changes to coverable lines.

❗ Current head afd118f differs from pull request most recent head 80dba4f. Consider uploading reports for the commit 80dba4f to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #619   +/-   ##
========================================
  Coverage    72.19%   72.19%           
========================================
  Files           94       94           
  Lines         9318     9318           
========================================
  Hits          6727     6727           
  Misses        2161     2161           
  Partials       430      430           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@FGYFFFF FGYFFFF force-pushed the feat/import_third_model branch from 44fd513 to 2634bc0 Compare February 17, 2023 09:19
@FGYFFFF FGYFFFF changed the title [WIP]feat(hz): use third model as "import" of handlers feat(hz): use third model as "import" of handlers Feb 17, 2023
@FGYFFFF FGYFFFF force-pushed the feat/import_third_model branch 3 times, most recently from dd06c30 to 22b7c76 Compare February 17, 2023 10:02
@FGYFFFF FGYFFFF force-pushed the feat/import_third_model branch from 22b7c76 to 80dba4f Compare February 20, 2023 02:37
@FGYFFFF FGYFFFF merged commit 318bea3 into cloudwego:develop Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants