Skip to content

Conversation

@benediktjohannes
Copy link
Contributor

@benediktjohannes benediktjohannes commented Jan 25, 2026

Add support for the DICOM (.dcm) MIME type by mapping it to application/dicom. Fixes and adds #144217

@benediktjohannes benediktjohannes requested a review from a team as a code owner January 25, 2026 03:09
@bedevere-app

This comment was marked as resolved.

@bedevere-app

This comment was marked as resolved.

@ZeroIntensity ZeroIntensity changed the title Add dicom support (medical imaging) to mimetypes, fixes #144217 gh-144217: Add dicom support (medical imaging) to mimetypes Jan 25, 2026
Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

I think something went wrong with the news entry. There should only be one of them.

@benediktjohannes
Copy link
Contributor Author

Thanks! Should now be fixed!

@benediktjohannes
Copy link
Contributor Author

Thanks for the Information! Now everything should be perfectly Fine and ready to merge, please Review @ZeroIntensity or @StanFromIreland , thanks!

@benediktjohannes
Copy link
Contributor Author

Important Note: I Hope that this works with just deleting (and before editing) the two (and also editing the third) news entrys which were too many (Hope this doesnt cause issues with the blurb bot, but I guess that this is just a creation Tool and doesnt have These linked, but please correct me if I am mistaken (as well thinking about the editing thing)).

@benediktjohannes
Copy link
Contributor Author

And I hope that this doesnt make Problems that the News Description (which is now consistent with some other) isnt exactly the same as in the other News File, but I think that this is Fine, but please correct me if thats a Problem.

@benediktjohannes
Copy link
Contributor Author

By the way, thanks for the re run of the Lint check

…1wVXH.rst

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
@benediktjohannes
Copy link
Contributor Author

Thanks for the Review! Changes are commited.

Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

Thanks!

@ZeroIntensity ZeroIntensity enabled auto-merge (squash) January 25, 2026 21:55
@ZeroIntensity ZeroIntensity merged commit 76d3ae7 into python:main Jan 25, 2026
47 checks passed
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