Skip to content
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

VorbisComment Recommendation and Chapter Support #969

Merged
merged 10 commits into from
Feb 12, 2025

Conversation

rgarber11
Copy link
Contributor

Currently, while cozy supports .opus files, it doesn't follow the xiph.org comment recommendations for tags or parse chapters. This pull request parses the author and reader per the above document, and also gets chapters if they conform to the ChapterExtension.

@winstonma
Copy link

winstonma commented Feb 10, 2025

Thanks I patched this on the master branch of cozy and it works flawlessly. Thanks a lot and hopefully it would be merged soon.

@rdbende
Copy link
Collaborator

rdbende commented Feb 11, 2025

I have no idea what issue Github has with the CI. I tried using the "Explain error" Copilot button, but it hallucinated garbage about dnf. Nothing unexpected, tbh. I don't know why the hype around it.

@rdbende
Copy link
Collaborator

rdbende commented Feb 12, 2025

Found the issue: flatpak/flatpak-github-actions#214
I'll just disable uploading the artifacts for now.

@rdbende rdbende merged commit 81b7e3f into geigi:master Feb 12, 2025
2 of 4 checks passed
@rdbende
Copy link
Collaborator

rdbende commented Feb 12, 2025

Thanks, @rgarber11!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants