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

863 adding swagger annotation support for enum and nested objects #865

Merged
merged 9 commits into from
Feb 14, 2019

Conversation

fahernandez
Copy link
Contributor

No description provided.

… object, repeated nested object and repeated primitive fields
@fahernandez
Copy link
Contributor Author

fahernandez commented Feb 11, 2019

Linked to issue 836

@codecov-io
Copy link

codecov-io commented Feb 12, 2019

Codecov Report

Merging #865 into master will decrease coverage by 0.05%.
The diff coverage is 31.81%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #865      +/-   ##
==========================================
- Coverage   53.01%   52.95%   -0.06%     
==========================================
  Files          39       39              
  Lines        3901     3907       +6     
==========================================
+ Hits         2068     2069       +1     
- Misses       1637     1642       +5     
  Partials      196      196
Impacted Files Coverage Δ
protoc-gen-swagger/genswagger/template.go 53.25% <31.81%> (-0.23%) ⬇️

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 c3923b1...a1d616b. Read the comment docs.

Copy link
Collaborator

@johanbrandhorst johanbrandhorst left a comment

Choose a reason for hiding this comment

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

LGTM, I'll give @achew22 24 hours to voice anymore opinions before merging.

@achew22 achew22 merged commit 45aec34 into grpc-ecosystem:master Feb 14, 2019
@achew22
Copy link
Collaborator

achew22 commented Feb 14, 2019

@fahernandez, thanks so much for finding a problem and reporting it but then going above and beyond to fix it for everyone! Really really great work. I look forward to doing it again soon.

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.

5 participants