gh-145917: Add MIME types for TTC and Haptics formats#145918
gh-145917: Add MIME types for TTC and Haptics formats#145918clin1234 wants to merge 12 commits intopython:mainfrom
Conversation
Added new MIME types for font collections and haptics.
|
@StanFromIreland, @hugovk, @benediktjohannes, @encukou: would any one of you review #146344, #146342, and this? I would like for all three to be merged in before the beta cutoff. Apologies if I sound a bit pushy. |
| * Add ``application/sql`` and ``application/vnd.sqlite3``. | ||
| (Contributed by Charlie Lin in :gh:`145698`.) | ||
| * Add ``image/jxl``. (Contributed by Foolbar in :gh:`144213`.) | ||
| * Add the following MIME types: |
There was a problem hiding this comment.
Could we maybe combine the list in one because both lists were authored by you?
There was a problem hiding this comment.
We then need to write „and gh …“
Misc/NEWS.d/next/Library/2026-03-13-14-23-33.gh-issue-145917.TooGKx.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Benedikt Johannes <benedikt.johannes.hofer@gmail.com>
Co-authored-by: Benedikt Johannes <benedikt.johannes.hofer@gmail.com>
Added new mimetypes for font and haptics.
No problem! Overall looks pretty good to me, nice contributions, thanks! |
benediktjohannes
left a comment
benediktjohannes
left a comment
There was a problem hiding this comment.
Overall looking pretty good to me! I did not look up the correctness of the applications and the entrys in official databases, but I can‘t approve the changes either because of permissions, so from what I saw when having a look on it, this LGTM, thanks!
|
Please also fix Bullet list ends without a blank line; unexpected unindent |
benediktjohannes
left a comment
benediktjohannes
left a comment
There was a problem hiding this comment.
Just one nitpick
| @@ -0,0 +1,8 @@ | |||
| * Add the following to ``mimetypes``: | |||
There was a problem hiding this comment.
Hmm, I think that in this news entry (where it is not 3.15.rst) the „“ is not necessary because this file itself is considered to be one entry in a list and therefore the „“ is then added automatically, but I may be mistaken. This also needs to be changed in the other PRs please. And also the contributor statement and potentially other elements which were moved to the right (due to this change) might need to get back again, please. But I might be mistaken. Is there any unit test suggesting this change?
There was a problem hiding this comment.
By the way: The rest looks quite good to me from a Short Overview, thank You very much for contributing!
|
Thank you very much, looks great to me! |
|
(I have not checked all mime types for correctness (e.g. typing, …) but just had a overview look on the PR in general, but based on my short review, I think that this is nice!) |
|
Unforetunately, we got (at least one) failing Check, but this first one seems to be unrelated, but please also have a look lateron on the potentially failing Checks whether they Are related or not because I didn’t Check on Small things Like Syntax |
2 participants
Uh oh!
There was an error while loading. Please reload this page.