-
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
V2: Update Makefile #1560
V2: Update Makefile #1560
Conversation
remove repeat net/http
* Add template action delimiter cli flag * Add delims to generator config and template Also adds tests using the "quote" test as a base. This has to have a custom Instance name or it will clash with the "quotes" one and panic since it will have registered two "swagger" instances in the package test. * Add testdata for custom delim flags Based on the "quote" testdata. * Add delims to the spec, with tests. Make sure we don't add delims if they are empty. This shouldn't be possible, but might as well be safe. * Go mod tidy and sum update * Make the CLI experience a bit cleaner * Revert go.mod and sum * Update readme
Signed-off-by: sdghchj <[email protected]>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## v2 #1560 +/- ##
==========================================
+ Coverage 77.67% 79.52% +1.85%
==========================================
Files 21 21
Lines 5528 5549 +21
==========================================
+ Hits 4294 4413 +119
+ Misses 1048 962 -86
+ Partials 186 174 -12
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
@Nerzal as a member, you can create a branch in swaggo/swag repository. You don't need to use a forked repo any more. |
Yeah, that is also definitely going to prevent problems with pushing tags to the wrong repo. I'll use this from now on |
Describe the PR
Merged master branch into v2 branch.
Also integrated the new delimiter feature into the v3 generator