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

Switch to better maintained go-charset fork (Closes: #134). #145

Closed
wants to merge 1 commit into from

Conversation

onlyjob
Copy link

@onlyjob onlyjob commented Mar 22, 2016

When this is merged it would be great to tag new formal release. Thanks.

@elazarl
Copy link
Owner

elazarl commented Mar 23, 2016

That would be great idea.

But I've already did that once, from the google code to rog pepe.

How can we find a repository that would be maintained in the long run?

@onlyjob
Copy link
Author

onlyjob commented Mar 23, 2016

Well, google code is gone and at the moment I suggest we switch to project that is currently best maintained. I doubt you can find guarantees that it will stay like this forever. One simply have to adapt to changes. Let's hope that new go-charset proves itself to be the best. :)

@onlyjob
Copy link
Author

onlyjob commented Mar 23, 2016

Also worth noticing that nobody can use rogpepe/go-charset due to licensing problems so there is a bigger problem than poorly maintained dependency...

@ErikPelli
Copy link
Collaborator

I completely removed this dependency in #580. Closing this PR.

@ErikPelli ErikPelli closed this Dec 20, 2024
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.

3 participants