Šarūnas Nejus
Šarūnas Nejus
> It looks like the Windows builds are broken because of the `reflink` issue. @snejus, feel free to review anyway. You can now rebase as the fix has been merged...
Everything looks good - there's only one bit that's left, see https://github.com/beetbox/beets/pull/5359#pullrequestreview-2246989763. What did we decide to do about the test setup?
> There seems to be some weird issue with test coverage. @snejus, any idea what's going on? I have a feeling that's because the workflow is being run from your...
@bal-e sorry I didn't realise this was ready! If you rebase the PR off master the coverage should run fine now.
@bal-e see #5479 where I did the same change which later got reverted since I discovered it indeed disabled PR checks 😅 > Is there any good way to disable...
For the future PRs, note that since you're a member of the organization you can push your work directly to a branch in this repo (`beetbox/beets`) - this issue does...
Accidentally stumbled upon something relevant: https://github.com/beetbox/beets/issues/1044
Superseded by #5890
> There seems to be some failures unrelated to my code in the lint checks Seems like this was caused by GitHub failing DNS resolution internally. Have re-ran the workflows...
Apologies - I forgot to merge this in 😅