Skip to content

add enum description #2561

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

Merged
merged 2 commits into from
Apr 18, 2021
Merged

add enum description #2561

merged 2 commits into from
Apr 18, 2021

Conversation

d0ngw
Copy link
Contributor

@d0ngw d0ngw commented Apr 17, 2021

Add the enum description to the generated swagger doc, it's useful when the enum is integer.

d0ngw added 2 commits April 17, 2021 20:42
Signed-off-by: d0ngw <agile.java@gmail.com>
Signed-off-by: d0ngw <agile.java@gmail.com>
@codecov
Copy link

codecov bot commented Apr 17, 2021

Codecov Report

Merging #2561 (03fd534) into master (d48ab3b) will decrease coverage by 0.09%.
The diff coverage is 63.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2561      +/-   ##
==========================================
- Coverage   81.57%   81.47%   -0.10%     
==========================================
  Files          54       54              
  Lines        8462     8503      +41     
==========================================
+ Hits         6903     6928      +25     
- Misses       1061     1072      +11     
- Partials      498      503       +5     
Impacted Files Coverage Δ
codescan/parser.go 74.96% <ø> (ø)
codescan/responses.go 71.92% <ø> (ø)
codescan/application.go 67.73% <47.61%> (-1.66%) ⬇️
codescan/parameters.go 71.57% <75.00%> (-0.01%) ⬇️
codescan/schema.go 67.29% <75.00%> (+0.20%) ⬆️
codescan/enum.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d48ab3b...03fd534. Read the comment docs.

@youyuanwu youyuanwu requested a review from casualjim April 17, 2021 18:13
@casualjim casualjim merged commit 131eb69 into go-swagger:master Apr 18, 2021
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.

2 participants