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

discogs-client now deprecated #3608

Closed
grawlinson opened this issue Jun 1, 2020 · 11 comments · Fixed by #3861
Closed

discogs-client now deprecated #3608

grawlinson opened this issue Jun 1, 2020 · 11 comments · Fixed by #3861
Labels
feature features we would like to implement

Comments

@grawlinson
Copy link
Contributor

As of today, the discogs-client package has been deprecated. This will become an issue when upstream breaks API compatibility.

This repository is no longer maintained. You can still use a REST client like Requests or other third-party Python library to access the Discogs REST API.

@sampsyo sampsyo added the needinfo We need more details or follow-up from the filer before this can be tagged "bug" or "feature." label Jun 2, 2020
@sampsyo
Copy link
Member

sampsyo commented Jun 2, 2020

That's too bad! Perhaps a volunteer is interested in forking it and attempting to maintain the library independently?

@grawlinson
Copy link
Contributor Author

Yeah, I'll make a fork as I use this quite extensively.

For now, are you OK to depend on the current package? I don't expect it to break compatibility anytime soon!

@sampsyo
Copy link
Member

sampsyo commented Jun 2, 2020

Yes, sounds good! No need to drop it if it's not currently causing problems. Thanks for being willing to adopt it! 😃

@grawlinson
Copy link
Contributor Author

grawlinson commented Jun 3, 2020

I've forked the current package here, feel free to update documentation/code pointing to it.

Also, I'd appreciate if feedback could be made regarding improvements to the code as-is. It'll take me a bit of time to look at the closed issues/PRs on discogs-client.

EDIT: The package is also on PyPi.

@stale
Copy link

stale bot commented Aug 2, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 2, 2020
@wisp3rwind wisp3rwind removed needinfo We need more details or follow-up from the filer before this can be tagged "bug" or "feature." stale labels Aug 2, 2020
@jhuldtgren
Copy link

is there timeline to get the code updated to use discogs_api vs discogs_client? I maintain the discogs_client port/package for OpenBSD and was looking to replace it with discogs_api but until the appropriate code is updated in beets that can't happen.

@grawlinson
Copy link
Contributor Author

A pull request will need to be proposed. I wish I had the time to do this, so if you could do that, it would be great.

It should be as simple as replacing all calls made to discogs_client with discogs_api.

Any further issues, I'll be happy to help.

@sampsyo sampsyo added the feature features we would like to implement label Dec 31, 2020
@sampsyo
Copy link
Member

sampsyo commented Dec 31, 2020

That would be great! And indeed; I don't think the change should be hard.

@pbnoxious
Copy link

I think all you need to do codewise is replace the imports, the API seems to have mostly stayed the same.

I actually did the changes and wanted to do a PR (here is the branch), but realized that there are now multiple forks, also one that keeps the package named discogs_client.
Using that one would likely only require to change the dependency discogs-client -> python3-discogs-client in setup.py (and maybe the documentation).

What are your thoughts? Which of the forks should be used?

@sampsyo
Copy link
Member

sampsyo commented Feb 12, 2021

Can we trust your judgment on that? Does any one fork look more promising than the others?

@grawlinson
Copy link
Contributor Author

Considering that the discogs community have converged on joalla/discogs_client rather than others (including my fork), it would be preferable to go with that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature features we would like to implement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants