FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-144217: Add dicom support (medical imaging) to mimetypes by benediktjohannes · Pull Request #144218 · python/cpython · GitHub

Repository navigation

gh-144217: Add dicom support (medical imaging) to mimetypes - #144218

Merged
ZeroIntensity merged 12 commits into
python:mainfrom
benediktjohannes:patch-3
Jan 25, 2026
Merged

ZeroIntensity merged 12 commits into
python:mainfrom
benediktjohannes:patch-3

Conversation

benediktjohannes commented Jan 25, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

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

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

This comment was marked as resolved.

This comment was marked as resolved.

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

ZeroIntensity left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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

Copy link
Copy Markdown
Contributor Author

Thanks! Should now be fixed!

Copy link
Copy Markdown
Contributor Author

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

Copy link
Copy Markdown
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)).

Copy link
Copy Markdown
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.

Copy link
Copy Markdown
Contributor Author

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

…1wVXH.rst

Co-authored-by: Peter Bierma <zintensitydev@gmail.com>

Copy link
Copy Markdown
Contributor Author

Thanks for the Review! Changes are commited.

ZeroIntensity left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks!

ZeroIntensity enabled auto-merge (squash) January 25, 2026 21:55
ZeroIntensity merged commit 76d3ae7 into python:main Jan 25, 2026
47 checks passed
thunder-coding pushed a commit to thunder-coding/cpython that referenced this pull request Feb 15, 2026
…thonGH-144218)

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Peter Bierma <zintensitydev@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL