Skip to content

Add response media types to openapi schema for images#221

Merged
vredchenko merged 1 commit intomainfrom
png-media-type-openapi-schema
Jan 29, 2026
Merged

Add response media types to openapi schema for images#221
vredchenko merged 1 commit intomainfrom
png-media-type-openapi-schema

Conversation

@d-j-hatton
Copy link
Contributor

Adds correct content description to a couple of endpoints in the autogenerated openapi schema.

I ran into a problem with the ruff pre-commit. It was complaining that it was receiving the --force-exclude flag multiple times. I haven't looked into it properly but my assumption is that the flag is baked into the hook in some way. Removing the argument from the pre-commit config fixed for me. I moved to v0.14.14 of the hook while I was at it, not sure if this was lower for a particular reason.

@d-j-hatton d-j-hatton requested a review from vredchenko January 29, 2026 09:58
@d-j-hatton d-j-hatton added the enhancement Minor improvements to existing functionality label Jan 29, 2026
@vredchenko
Copy link
Collaborator

I can't recall a specific reason for why the version was lower, if there was an important one CI will tell us. Also FYI if you need to bypass local git hooks: git commit --no-verify (certainly for a change such as this one bypassing is fine). I'll look into --force-exclude 👍

@vredchenko vredchenko force-pushed the png-media-type-openapi-schema branch from d1977d8 to c209cab Compare January 29, 2026 11:24
@vredchenko vredchenko merged commit 3562e06 into main Jan 29, 2026
4 checks passed
@vredchenko vredchenko deleted the png-media-type-openapi-schema branch January 29, 2026 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Minor improvements to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants