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

fix importing with go mod #146

Closed
wants to merge 1 commit into from
Closed

Conversation

tych0
Copy link

@tych0 tych0 commented Mar 7, 2019

The repositories github.com/mattn/go-colorable and
github.com/mattn/go-isatty have moved to using go modules; using gopkg.in
to import them breaks:

golang/go#30636

Let's use the real URLs with hashes.

Signed-off-by: Tycho Andersen [email protected]

The repositories github.com/mattn/go-colorable and
github.com/mattn/go-isatty have moved to using go modules; using gopkg.in
to import them breaks:

golang/go#30636

Let's use the real URLs with hashes.

Signed-off-by: Tycho Andersen <[email protected]>
@x1unix
Copy link

x1unix commented Apr 29, 2019

@cheggaaa can you review this PR?

@cheggaaa
Copy link
Owner

Currently, progress bar does not work with latest go-runewidth (#142). I hope to fix it soon...

@udhos
Copy link

udhos commented May 15, 2019

I need this change as well.

If you add the support for Modules, you can pin down the dependency to a specific go-runewidth version:

go get github.com/mattn/go-runewidth@version

@jsnfwlr
Copy link

jsnfwlr commented Jul 1, 2019

I gave up hope with this PR - it has been too long. I forked my own copy of this repo and made the changes myself.

https://github.com/TheBrazenGeek/progbar

@cheggaaa
Copy link
Owner

cheggaaa commented Jul 1, 2019

Sorry for the late reaction. I could not decide how to do it correctly in order to save both versions of the library and not break backward compatibility.
Please look at my PR #150

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants