-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Support parse definition in vendor, for issues/242 #331
Conversation
Codecov Report
@@ Coverage Diff @@
## master #331 +/- ##
==========================================
+ Coverage 87% 87.02% +0.02%
==========================================
Files 7 7
Lines 1277 1279 +2
==========================================
+ Hits 1111 1113 +2
Misses 103 103
Partials 63 63
Continue to review full report at Codecov.
|
@@ -59,6 +61,10 @@ func main() { | |||
Value: "./docs", | |||
Usage: "Output directory for al the generated files(swagger.json, swagger.yaml and doc.go)", | |||
}, | |||
cli.BoolFlag{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think?
@easonlin404
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good.
LGTM |
Thx! |
@a180285 @pei0804 would you help add |
@a180285 |
I forget to update the doc before you notice me. I can update docs tomorrow. If you are urgent to update the doc, you can help to update it. |
OK I write. |
Thanks |
Describe the PR
Support parse definition in vendor, for issues/242
#242
Relation issue
#242
Additional context
I found that this tool can work well, expect for parsing this file: https://github.com/swaggo/swag/blob/master/operation.go
Which file is in this repo~~ So I ignored this repo when parse vendor folder