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

Add m4b as a type of audio/mp4 #357

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

dhouck
Copy link

@dhouck dhouck commented Jan 15, 2025

I made this change on the GitHub website; if there are any tests to run I did not see information about them and did not run them.

This is commonly used for audiobooks
@wesleytodd
Copy link
Member

Bookmarkable AAC files may carry the extension m4b

From the linked doc

Add a comma missed in previous commit
@dhouck
Copy link
Author

dhouck commented Jan 16, 2025

Oops. I kept double-checking the lines I added for JSON errors but forgot to check the line above. Iʼm still having computer issues and canʼt run complicated tests right now but jq can read the file now and it canʼt read the old one, so that probably works.

@wesleytodd
Copy link
Member

It seems valid to me, and the coverage is fickle and has been removed in a few of our repos. It is not necessary and once the window for reviews (I think 2 days) is up we can merge this.

@dhouck
Copy link
Author

dhouck commented Jan 16, 2025

Bookmarkable AAC files may carry the extension m4b

From the linked doc

Yep, thatʼs the part Iʼm using as a citation. If that isnʼt good Iʼm sure I could find other sources.

Copy link
Member

@wesleytodd wesleytodd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats good, just wanted to save people the click.

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