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

fix: enums with paramType=body got parsed incorrectly #1625

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

hohobilly
Copy link
Contributor

Describe the PR
Attempt to fix the issue #1608

Relation issue
#1608

Additional context
Adding a test case with the commentLine expression similar to the one suggested in the issue.

@hohobilly hohobilly changed the title fix: enums with paramType=body got parse outside schema attr fix: enums with paramType=body got parsed incorrectly Jul 17, 2023
@codecov
Copy link

codecov bot commented Jul 18, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (fe971d2) 83.72% compared to head (6654440) 83.74%.

❗ Current head 6654440 differs from pull request most recent head 6d6791c. Consider uploading reports for the commit 6d6791c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1625      +/-   ##
==========================================
+ Coverage   83.72%   83.74%   +0.01%     
==========================================
  Files          19       19              
  Lines        3767     3771       +4     
==========================================
+ Hits         3154     3158       +4     
  Misses        524      524              
  Partials       89       89              
Impacted Files Coverage Δ
operation.go 95.06% <100.00%> (+0.02%) ⬆️

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

@ubogdan
Copy link
Contributor

ubogdan commented Jul 18, 2023

@hohobilly Thanks for your contribution!

@ubogdan ubogdan merged commit 4536bf2 into swaggo:master Jul 18, 2023
10 checks passed
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.

None yet

2 participants