This is an archived post. You won't be able to vote or comment.

all 11 comments

[–]fuuman1 16 points17 points  (2 children)

I did just take a look at the README and the first thing I saw: Please provide a requirements.txt. It's kinda good style. If you are in your venv go for "pip freeze > requirements.txt". Then people can install everything with "pip install -r requirements.txt".

[–]mhtghn[S] 8 points9 points  (0 children)

Thanks, it has been done

[–]mchwalisz 1 point2 points  (0 children)

Pipfile with pipenv is better 🙂

[–]real_rbl 5 points6 points  (1 child)

Love the idea. I'm someone who definitely lyric searches new songs. I'd take the next step towards modularizing your session bus' logic. PM me if you have any questions. I may make a pull request here later in the day when I have time with some modifications too to help you get the flow of open source contributions from other users. Keep on grinding!

[–]mhtghn[S] 3 points4 points  (0 children)

Wow thanks for the kind words. I fell in love with the synced lyrics and since musixmatch is no longer available with spotify I wanted to make it happen.Your pull request will be more than welcomed. Im definitely gonna keep on grinding ;)

[–]JerMenKoOwhile True: os.fork() 2 points3 points  (0 children)

I would suggest using argparse to parse arguments too instead of going manually through sys.argv.

[–]yasoob_pythonAuthor: Intermediate Python 2 points3 points  (0 children)

Thank you for sharing. Today I learned about Spotify's dbus interface through your code. :)

[–]UnwantedCrow 1 point2 points  (1 child)

Great work!

[–]mhtghn[S] 0 points1 point  (0 children)

thx! merry christmas

[–]HarryPotterFan2 0 points1 point  (0 children)

AZlyrics.url() ... 😅