Merge lp:~charclo-michael/entertainer/bug-228033-2 into lp:~entertainer-releases/entertainer/devel

Proposed by Charclo
Status: Merged
Merge reported by: Paul Hummer
Merged at revision: not available
Proposed branch: lp:~charclo-michael/entertainer/bug-228033-2
Merge into: lp:~entertainer-releases/entertainer/devel
To merge this branch: bzr merge lp:~charclo-michael/entertainer/bug-228033-2
To post a comment you must log in.
Revision history for this message
Paul Hummer (rockstar) wrote :

 M src/frontend/gui/tabs/tab.py
 M src/frontend/media_player.py
 M src/frontend/medialibrary/music.py
 M src/utils/lyrics_downloader.py
Text conflict in src/frontend/media_player.py
Text conflict in src/frontend/medialibrary/music.py
2 conflicts encountered.

228. By Charclo

resolved conflicts

229. By Charclo

Merged trunk and resolved new conflicts

Revision history for this message
Paul Hummer (rockstar) wrote :

Your branch looks good, and it could be even better by adding some tests. Lyrics fetching has been a sore spot that causes a lot of headache, and tests would help us catch problems early on. Please add those tests and I'll merge the branch in.

230. By Charclo

Added a unittest

Revision history for this message
Paul Hummer (rockstar) wrote :

I branched this branch and completed the work on the final unit test. It has been proposed for merging, and I will be merging it soon. Thanks for your work!

Subscribers

No one subscribed via source and target branches