-
Notifications
You must be signed in to change notification settings - Fork 544
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
[Feature Request]auto gen swagger comment for handler #766
Comments
Good idea, I'll add a responsive comment feature after I've researched it clearly |
comment := "// " + m.Name + "\n"
comment += "// @Summary " + text + "\n"
comment += "// @Description \n"
comment += "// @Tags " + strings.Join(strings.Split(m.OutputDir, "/"), " ") + "\n"
if m.Serializer == "JSON" {
comment += "// @Accept json\n// @Produce json\n"
}
comment += "// @Response 200 {object} " + m.ReturnTypeName + "\n"
comment += "// @Router " + m.Path + " [" + m.HTTPMethod + "]"
m.Comment = comment I find some pain points:
|
good idea~ |
I have been working around on another approach, an example here: https://github.com/ringsaturn/protoc-gen-go-hertz/tree/main/_example/hertz-blog-service |
👍🏻,可以联系我下,加入到社区开发者群中。 |
Is your feature request related to a problem? Please describe.
when using idl for generate code, we currently add some simple comments for each handler
hertz/cmd/hz/generator/handler.go
Lines 247 to 261 in 0d8b98b
but we have fully information for this endpoint, e.g. http method, serializer type, url path, request params, response params, and we have full info for request struct and response struct, like which field is used in path/query/body. So we can add a flag for hz to control generating swagger style comment for each method.
previous relates issues #28, #538, #558, #692
Describe the solution you'd like
In HttpMethod.InitComment, generate swagger style comment for using https://github.com/hertz-contrib/swagger
Describe alternatives you've considered
maybe we could provider a template like layout and package for the swagger comments.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: