Skip to content
This repository was archived by the owner on Mar 19, 2024. It is now read-only.

Update Async-Http-Client to version 1.0.0 #4

Conversation

Andrea-Scuderi
Copy link
Member

@Andrea-Scuderi Andrea-Scuderi commented Nov 6, 2019

  • Add new lambda handlers to rely directly on SwiftNIO 2.0 EventLoop.
public typealias SyncCodableNIOLambda<Event: Decodable, Response: Encodable> = (Event, Context) throws -> EventLoopFuture<Response>
public typealias SyncDictionaryNIOLambda = ([String: Any], Context) throws -> EventLoopFuture<[String: Any]>
public typealias AsyncCodableNIOLambda<Event: Decodable, Response: Encodable> = (Event, Context, @escaping (Result<Response, Error>) -> Void) -> Void
public typealias AsyncDictionaryNIOLambda = ([String: Any], Context, @escaping (DictionaryResult) -> Void) -> Void

@Andrea-Scuderi Andrea-Scuderi force-pushed the feature/async-http-client-1_0_0 branch from 1b93636 to c2289e1 Compare November 11, 2019 18:06
@codecov-io
Copy link

codecov-io commented Nov 11, 2019

Codecov Report

Merging #4 into master will decrease coverage by 1.2%.
The diff coverage is 97.1%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   99.01%   97.81%   -1.21%     
==========================================
  Files           2        3       +1     
  Lines         102      229     +127     
==========================================
+ Hits          101      224     +123     
- Misses          1        5       +4
Impacted Files Coverage Δ
...es/LambdaSwiftSprinterNioPlugin/LambdaApiNIO.swift 98.57% <100%> (+0.02%) ⬆️
Sources/LambdaSwiftSprinterNioPlugin/Utils.swift 97.77% <93.33%> (-2.23%) ⬇️
...ambdaSwiftSprinterNioPlugin/NIOLambdaHandler.swift 97.36% <97.36%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 47b8b9e...8ca41c8. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants