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

lib: improve help information for api parameters #909

Merged
merged 1 commit into from
Dec 29, 2023
Merged

Conversation

JacksonTian
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Dec 26, 2023

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (39556a3) 82.66% compared to head (0db0e7c) 82.55%.

Files Patch % Lines
openapi/library.go 80.76% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
- Coverage   82.66%   82.55%   -0.12%     
==========================================
  Files          47       47              
  Lines        3860     3869       +9     
==========================================
+ Hits         3191     3194       +3     
- Misses        539      544       +5     
- Partials      130      131       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -81,6 +115,9 @@ func GetProductName(language, code string) (name string, err error) {

products := new(ProductSet)
err = json.Unmarshal(content, &products)
if err != nil {
return
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里不需要【return nil, err】吗


func TestGetAPIDetail(t *testing.T) {
api, err := GetAPIDetail("en", "ecs", "DescribeRegions")
assert.Nil(t, err)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里的err一直为nil

@JacksonTian JacksonTian merged commit 53b0c35 into master Dec 29, 2023
5 checks passed
@JacksonTian JacksonTian deleted the api_help branch December 29, 2023 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants