Prefer synced lyrics, options to allow only plain/synced

[jmaximusix]

• Prefer synced lyrics, but allow plaintext as fallback (default behaviour) (fixes #45 )
• to not break backwards compatibility, the --allow-plain option still exists as an alias for the new (more concise) --plaintext-only
• new --synced-only option
• Documented behaviour of --enhanced: If no word-by-word sync is available, synced lyrics are used as a fallback. (Behaviour didn't change, but updated README for clarity)
• if no translation is found it raises an exception (that is caught above). This is in my opinion better than checking if the translation worked later
• small fix / enhancement: added __str__ method to LRCProvider, to avoid repeated provider.__class__.__name__()

[jmaximusix]

uhmmm... well that's unfortunate xD I just removed the print statement, now apparently they couldn't reach deezer? Can you somehow restart the tests or do they only run on commits?

[moehmeni]

Seems Deezer doesn't work anymore lol. I wanted to comment out Deezer calls in search function and also put a help comment in readme but I don't know what happens if I change the main branch and your updates are based on previous version. I appreciate it if you can do it (also comment the Deezer test) so the tests pass and I will merge the changes.

[jmaximusix]

Ok, I used black to reformat, and changed the functions back to lambdas.

Also commented out Deezer. Are we sure yet whether this broke permanently (due to an api change of theirs or sth) or if it's only temporarily down? Edit: Just saw you opened an issue, and based on the response seems like an api change you're right

If there's anything else you want to change before merging, let me know :)

[moehmeni]

Great. Thanks again!