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
Add a way to convert HTTP methods from/to String #976
Conversation
Can one of the admins verify this patch? |
4 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
6ee8874
to
4ef16ff
Compare
@swift-nio-bot test this please |
@tomerd I reverted back to 2018 for the generated-linux tests and I'll propose a patch soon to bump all the copyrights or something like that. |
@swift-nio-bot test this please |
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.
Thank you! I'm happy with this. Two questions for @Lukasa though:
- do we want the name to be
rawValue
? - should be make the constructor fallible? Just in case someone does something clearly illegal like the empty string or something with a space. We don't even need to check the validity today but maybe we should make it fallible just so we can in the future?
Motivation: apple#701 Modifications: * Add `init(rawValue:)` * Add `rawValue` * Add tests in HTTPTypesTest.swift Result: Users no longer have to write their own conversion methods.
@swift-nio-bot add to whitelist |
@swift-nio-bot test this please |
Motivation:
#701
Modifications:
init(rawValue:)
to HTTPMethodrawValue
property to HTTPMethodResult:
Users no longer have to write their own conversion methods.