DamienCassou commented on issue mpdel/mpdel#34

Some suggestions: - I would revert the `if` condition to exit early and get rid of the `progn`: ```emacs (if (seq-empty-p songs) (user-error "Sorry, ...") …

2 months ago

DamienCassou deleted branch unixsockets from mpdel/libmpdel

2 months ago

DamienCassou commented on issue mpdel/mpdel#33

I don't have any suggestion to edit metadata of a song. I suggest you experiment with something and when you are satisfied you show me what you have done. I think the simplest way would be to make…

2 months ago

DamienCassou commented on pull request mpdel/mpdel#32

Thank you very much!

3 months ago

DamienCassou merged pull request mpdel/mpdel#32

[#31] Implement navigel-delete for stored-playlists

3 months ago

DamienCassou commented on issue mpdel/mpdel#33

Thanks for the picture. This looks like a much improved version of mpdel-song-mode. One thing we could do is to improve the mpdel-song-mode to look more like that. Another thing is to make this…

3 months ago

DamienCassou commented on issue mpdel/mpdel#33

By "edit", do you mean "edit metadata"? It could be really fun! Some ideas: - look at emms (e.g., through this video https://emacsconf.org/2020/talks/05/, starting at 03:00) - through the…

3 months ago

DamienCassou suggested changes for mpdel/mpdel#32

Can you please squash the 2 commits into 1 and make sure the commit doesn't change mpdel-playlist.el?

3 months ago

DamienCassou deleted branch stored_playlists from mpdel/libmpdel

3 months ago

DamienCassou commented on pull request mpdel/libmpdel#28

Awesome. Thank you.

3 months ago

DamienCassou merged pull request mpdel/libmpdel#28

[#27] Implement libmpdel-stored-playlists-delete

3 months ago

DamienCassou commented on pull request mpdel/libmpdel#28

Final step: can you please squash your 2 commits into just one?

3 months ago

DamienCassou suggested changes for mpdel/mpdel#32

Thank you!

3 months ago

DamienCassou commented on pull request mpdel/mpdel#32

`mpdel-browser-mode` stuff should probably be in `mpdel-browser.el`.

3 months ago

DamienCassou commented on issue mpdel/mpdel#31

You are welcome. Thank you for your work.

3 months ago

DamienCassou suggested changes for mpdel/libmpdel#28

Good job. Please handle the small feedback and rebase your PR on top of latest master. Then we can merge.

3 months ago

DamienCassou commented on pull request mpdel/libmpdel#28

Can you please delete this useless line? Only one is necessary.

3 months ago