Skip to content

Conversation

gforcada
Copy link

@gforcada gforcada commented Oct 11, 2025

Hi,

while doing some contributions for Plone (on Products.MimetypesRegistry distribution) I drifted away and eventually noticed that TOML files are not recognized by mimetypes module.

I followed this commit as in what to change.

Compared to the commit referenced above, I decided to skip updating Doc/whatsnew/3.15.rst as it feels way too small of a change to mention it.

Let me know if it should still be updated nonetheless.

@gforcada gforcada requested a review from a team as a code owner October 11, 2025 15:49
Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

Looking at previous additions, can you please add this to the What's New too.

'.aiff' : 'audio/x-aiff',
'.ra' : 'audio/x-pn-realaudio',
'.wav' : 'audio/vnd.wave',
'.weba' : 'audio/webm',
Copy link
Member

Choose a reason for hiding this comment

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

Why this unrelated change?

Copy link
Author

Choose a reason for hiding this comment

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

As the commit explains, they were not sorted. The font/ related mimetypes had this audio/ mimetype in between.

Copy link
Member

Choose a reason for hiding this comment

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

In general please don't mix in changes, it is more difficult to review as a result.

…nCakj.rst

Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants