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

Refactor to use scylladb/termtables #101

Merged
merged 1 commit into from Apr 9, 2020
Merged

Conversation

creekorful
Copy link
Contributor

@creekorful creekorful commented Apr 9, 2020

Hello @araddon,

While I was packaging your library for Debian,
I have noticed that one of your dependency is not anymore available.

As you may all already know, apcera/termtables has come to a end.

This commit refactor to use the fork from scylladb.
ScyllaDB is an organization, so there's less chance that their
own fork goes offline too.

This PR fix #90 and is a lighter fix than the one proposed in #89 (vendoring termtables) because there's nothing else than repairing the library in this PR.

I don't know what do you think about this. Just let me know If you have another solution.
I require your library to package another one, so this fix is really important for me, and I'm willing to take time to do it in another way if you prefer.

Cheers

As you may all already know, apcera/termtables has come to a end.
This commit refactor to use the fork from scylladb.

ScyllaDB is an organization, so there's less chance that their
own fork goes offline too.
@codecov-io
Copy link

codecov-io commented Apr 9, 2020

Codecov Report

Merging #101 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #101   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          899       899           
=========================================
  Hits           899       899           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0fb0a47...acba2cf. Read the comment docs.

@troyspencer
Copy link
Contributor

👍

@araddon
Copy link
Owner

araddon commented Apr 9, 2020

awesome, thank you!

@araddon araddon merged commit d820a61 into araddon:master Apr 9, 2020
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.

github.com/apcera/termtables is no longer available, breaking main.go and travis CI
4 participants