Skip to content

Handle additional HTTP methods, e.g. QUERY - #957

Merged
czechboy0 merged 1 commit into
apple:mainfrom
czechboy0:hd-handle-more-http-methods
Sep 29, 2026
Merged

czechboy0 merged 1 commit into
apple:mainfrom
czechboy0:hd-handle-more-http-methods

Conversation

@czechboy0

Copy link
Copy Markdown
Contributor

Motivation

The TrainTravel reference OpenAPI doc has added a "SUBSCRIBE" method, which surfaced that
right now we generate non-compiling code in that case.

Modifications

Fixed that and also added QUERY support while I was in the code.

Result

TrainTravel and other OpenAPI docs will start compiling again.

Test Plan

Adapted tests.

@simonjbeaumont simonjbeaumont left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@czechboy0 czechboy0 added the 🔨 semver/patch No public API change. label Sep 29, 2026
@czechboy0
czechboy0 merged commit c46ba49 into apple:main Sep 29, 2026
48 of 50 checks passed
@czechboy0
czechboy0 deleted the hd-handle-more-http-methods branch September 29, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 semver/patch No public API change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants