-
Notifications
You must be signed in to change notification settings - Fork 231
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
Change import path of cli package #261
Conversation
I also have same problem. |
This PR is merged to my forked project. |
|
@stanxing Please check your branch :) |
Look at the output above, anything wrong with me? |
c94923e
to
67bacda
Compare
Looks good to me. Any plan to merge this?? |
I think this project is not updated recently. I'm using forked project now. My repository (master branch) contains this PR already. |
This PR has an issue below.
I change the name from "github.com/urfave/cli/v2" to "github.com/urfave/cli". Does anyone know the solution? |
I understand. |
Same here |
@asoseil Please have a look at this |
I submitted new PR #265. I think new one is a better fix than this PR. |
Overview
Issue is here.
Modifications
I changed import path of urfave/cli like below and fixed it.
Before: "gopkg.in/urfave/cli.v2"
After: "github.com/urfave/cli/v2"
Test result