Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Fix mixed path components #491

Merged
merged 2 commits into from
Dec 15, 2023

Conversation

czechboy0
Copy link
Contributor

Motivation

Fixes #490.

Modifications

Refactors the logic for composing path parameters on the client to support mixed components, which contain both a constant and a variable part (or two variable parts, etc).

Result

Support more OpenAPI docs.

Test Plan

Expanded a snippet test to cover this.

@czechboy0 czechboy0 merged commit 50cf5ac into apple:main Dec 15, 2023
9 checks passed
@czechboy0 czechboy0 deleted the hd-fix-mixed-path-components branch December 15, 2023 11:07
@czechboy0 czechboy0 added the 🔨 semver/patch No public API change. label Dec 15, 2023
@czechboy0
Copy link
Contributor Author

czechboy0 commented Dec 15, 2023

Fixed in 1.1.0.

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.

Support mixed path parameters (part constant, part variable) in client requests
2 participants