Sebastian Mohr
Sebastian Mohr
What do you mean? There are two PRs. This one is based on #86, which is a minimal refactoring using the same build tools as before.
@JOJ0 I have seen this, just had not that much time recently. Will hopefully have a look this weekend, I think we can cherry pick most things from here anyways....
This should also be mostly ready. I do not like that we have to prefix every command with `uv run` in the workflows tho. Would be nice to have a...
You have seen that there is already a PR for this? Also is there a reason to ascii decode the query? As far as I can see this is not...
Was added in a commit in part of the initial spotify plugin implementation. The reason for adding is explained in a comment, I don't think it is up to date...
I think https://github.com/beetbox/beets/issues/5189 and maybe https://github.com/beetbox/beets/issues/4514 could be related.
As there wasn't any commit from @dhruvravii for some time on this. I just added a config option to enable the legacy behavior and adjusted the tests to check for...
@beetbox/maintainers Could we get another look and some feedback here? I think this is an easy win.
> The idea of searching with the original unicode string and then falling back on bad (zero) results to the ascii strategy is worth considering! Would that be difficult to...
It seems like windows tests are broken at the moment. Will wait with the merge until they are fixed :)