search

robert-alfaro / genius-lyrics

Home Assistant custom component for fetching song lyrics from Genius.com
23 stars 7 forks source link

Fix #5: Handle Genius search failure / song not found #6

Closed airdrummingfool closed 5 years ago

airdrummingfool commented 5 years ago

Here's an updated PR that should merge cleanly. I had a feeling #5 was going to have some conflicts, sorry about that!

robert-alfaro commented 5 years ago

Looking more closely, I deleted the wrong section in the merge conflict. I did something I swore never to do..I used the web conflict resolver :scream:

airdrummingfool commented 5 years ago

Hahaha, happens to us all :) I imagine that's a good rule-of-thumb... I've never tried github's conflict resolver, but then again, I'm quite partial to the one I use (p4merge).