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

Compatibility with entirety of Genius; Song Search based on Title + Artist #10

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

yuichitlo
Copy link

In order to accommodate Genius' growing database and searches outside of rap and rock, everything has been generalized to http://genius.com and searches can be refined by the combination of title and artist information. This works much better as you filter out songs of the same title within a genre and are more likely to get a correct result.

I also thought renaming the project would make sense! Thanks for creating this API for working with Genius!

@kenshiro-o
Copy link
Owner

@yuichitlo Sorry for the super late getting back! It seems I took too long a break from this project.
Thanks for contributing!

I have had a quick skim through your code. It seems you commented out a few portions of code and moved stuff from my constants directly inline within the actual code logic. I am honestly not a big fan of it but would be keen to read more about your motivation.

By the way your changes seem to have broken the build so I would not be able to merge unless the build is fixed.
However, I will keep reading your changes and review.

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.

None yet

2 participants