-
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
fix: fix cli dependencies #603
Conversation
Codecov Report
@@ Coverage Diff @@
## master #603 +/- ##
==========================================
- Coverage 85.94% 85.82% -0.12%
==========================================
Files 7 7
Lines 1693 1693
==========================================
- Hits 1455 1453 -2
- Misses 149 150 +1
- Partials 89 90 +1
Continue to review full report at Codecov.
|
@swaggo/swaggo-core-team please help review it. |
I will jump to v1.6.5 if there are no problems. |
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.
LGTM
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.
LGTM
Previously we used clone
swag/cli
instead ofurfave/cli
as not compatible with original v1 codes. Nowurfave/cli
v2 has a different package name. so that we can still use v1.In the meantime, also drop go1.10 supporting for compatible with
urfave/cli
.Also fix #598