-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TaglibSharp 2.2.0 has been released #10
Comments
@decriptor I don’t think we mind blitzing the package, but the main reason we started was due to long periods of unresponsiveness on the main repo. This was causing issue with our user base and implementation that we needed to solve. Do you feel confident that when issues come up that they can/will be addressed quickly? |
@Qstick Funny enough you caught me on vacation. I think now that things are a bit more cleaned up and simplified I'm hoping to have more regular responsive release. If you need a PR merged and a release done quickly I think I'm better able to do that now. |
@decriptor If I PR #12 upstream any chance of a merge + release in the next few days? Turns out lidarr is corrupting OPUS files on tag write 😬 |
@ta264 Let me see if I can knock something out this weekend. |
Thanks! You can hit me up on the lidarr discord if you want to discuss anything |
@ta264 Would you be ok creating a PR against mono/taglib-sharp and create a test? If not, I'll create a PR and try to put together a unit test. Let me know. |
I was testing locally by calling out to opusinfo to verify integrity. Would that be acceptable or do you want it to be totally self contained? |
@ta264 It is so that CI can run the test and make sure we don't regress when new PRs come in. Essentially making sure this bug isn't regressed again. |
If you all are willing it'd be nice to close this repo and just work on the upstream version. I believe I've merged all of the changes at this point and published a new nuget package.
https://github.com/mono/taglib-sharp/releases/tag/TaglibSharp-2.2.0
The text was updated successfully, but these errors were encountered: