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

Support parse definition in vendor, for issues/242 #331

Merged
merged 2 commits into from Mar 21, 2019

Conversation

a180285
Copy link
Contributor

@a180285 a180285 commented Mar 20, 2019

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

@codecov-io
Copy link

codecov-io commented Mar 20, 2019

Codecov Report

Merging #331 into master will increase coverage by 0.02%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            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
Impacted Files Coverage Δ
operation.go 84.63% <ø> (ø) ⬆️
gen/gen.go 68.88% <100%> (+0.7%) ⬆️
parser.go 88.77% <80%> (+0.01%) ⬆️

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 93d87f4...e07e78e. Read the comment docs.

@@ -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{
Copy link
Member

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

Copy link
Member

Choose a reason for hiding this comment

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

It's good.

parser.go Show resolved Hide resolved
@pei0804
Copy link
Member

pei0804 commented Mar 21, 2019

LGTM

@pei0804 pei0804 merged commit b6a3401 into swaggo:master Mar 21, 2019
@pei0804
Copy link
Member

pei0804 commented Mar 21, 2019

Thx!

@easonlin404
Copy link
Member

@a180285 @pei0804 would you help add parseVendor flag doc in https://github.com/swaggo/swag#swag-cli?

@pei0804 pei0804 mentioned this pull request Mar 21, 2019
@pei0804
Copy link
Member

pei0804 commented Mar 21, 2019

@a180285
Can you? or I write?

@a180285
Copy link
Contributor Author

a180285 commented Mar 21, 2019

@pei0804

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.

@pei0804
Copy link
Member

pei0804 commented Mar 21, 2019

OK I write.

@a180285
Copy link
Contributor Author

a180285 commented Mar 21, 2019

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants