-
Notifications
You must be signed in to change notification settings - Fork 114
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
[Proposal] SOAR-0012 Generate enums for server variables #629
base: main
Are you sure you want to change the base?
[Proposal] SOAR-0012 Generate enums for server variables #629
Conversation
Whoops, SOAR-0011 is already taken, please use SOAR-0012 :) #626 |
Looks good overall, almost ready to go into review. Could you just slightly elaborate on the behavior with the feature flag enabled/disabled in the API stability section? Right now it sounds like it could be API breaking even for folks who don't opt-in, which isn't the case. |
9abfdfd
to
663dbcd
Compare
Fixed :) |
Done :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two more nits, after that I'm happy to send this into review.
Sources/swift-openapi-generator/Documentation.docc/Proposals/SOAR-0012.md
Outdated
Show resolved
Hide resolved
Sources/swift-openapi-generator/Documentation.docc/Proposals/SOAR-0012.md
Outdated
Show resolved
Hide resolved
This proposal is now In Review, forums thread: https://forums.swift.org/t/proposal-soar-0012-generate-enums-for-server-variables/74737 |
Sources/swift-openapi-generator/Documentation.docc/Proposals/SOAR-0012.md
Outdated
Show resolved
Hide resolved
8960a5a
to
777db9c
Compare
Motivation
As requested by @czechboy0 in #618 I have created this proposal for community feedback.
Modifications
Added the proposal.
Also fixed a typo in the document for the proposal process.
Result
N/A
Test Plan
N/A