-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Moving TransferState out of http folder #1369
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
Conversation
7388250
to
7ce9346
Compare
This is just a file movement intended for work coming up in a future PR. @swift-ci please test and merge |
@swift-ci please test and merge |
1 similar comment
@swift-ci please test and merge |
Build failure again. Looks like a CI problem.
|
@pushkarnk Looking into it. |
@swift-ci please test and merge |
@pushkarnk It should be fixed now |
Thanks @shahmishal ! I appreciate the prompt response :) |
Thank you @shahmishal |
TransferState has 1-on-1 relationship with EasyHandle and is independent of protocol.Hence it has been moved out of the http folder.In the next PR, this code will be aligned under NativeProtocol instead of HTTPURLprotocol which can be used across different protocols.
@alblue , I understand that this PR alone will not make much sense as the code is still an extension of HTTPURLProtocol.To address #1355 ,I have raised PR for the movement of the file.The code under this PR will be renamed to
_NativeProtocol
in the next PR.