Skip to content

Add non-array get support, update doc#9

Merged
sandmman merged 2 commits intodevelopfrom
add_nonarrayget_and_doc
Jan 10, 2018
Merged

Add non-array get support, update doc#9
sandmman merged 2 commits intodevelopfrom
add_nonarrayget_and_doc

Conversation

@tunniclm
Copy link
Copy Markdown
Collaborator

This PR adds support for calling get on a route without providing an ID and getting a non-array Codable returned. This is the client counterpart to PR Kitura/Kitura#1176 and is also dependent on Kitura/KituraContracts#4.

@sandmman
Copy link
Copy Markdown
Contributor

sandmman commented Jan 5, 2018

@tunniclm I think this should be ready to go if the package.swift gets updated (Kitura: 2.1.x & SwiftyRequest: 1.0.x)

@tunniclm tunniclm force-pushed the add_nonarrayget_and_doc branch from 8018e22 to aea4890 Compare January 5, 2018 16:56
@tunniclm tunniclm changed the base branch from master to develop January 5, 2018 16:56
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #9 into develop will increase coverage by 2.81%.
The diff coverage is 83.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop       #9      +/-   ##
===========================================
+ Coverage    64.04%   66.85%   +2.81%     
===========================================
  Files            5        2       -3     
  Lines          712      175     -537     
  Branches        31        0      -31     
===========================================
- Hits           456      117     -339     
+ Misses         256       58     -198
Flag Coverage Δ
#KituraKit 66.85% <83.33%> (+2.81%) ⬆️
Impacted Files Coverage Δ
Sources/KituraKit/Client.swift 69.51% <83.33%> (-2.92%) ⬇️
Sources/KituraKit/RequestErrorExtension.swift 27.27% <0%> (-8.45%) ⬇️

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 8ac3c6b...aea4890. Read the comment docs.

@tunniclm
Copy link
Copy Markdown
Collaborator Author

I'd rather do the SwiftyRequest update separately unless there's a reason why it is needed for this PR that I'm missing.

@sandmman
Copy link
Copy Markdown
Contributor

None that I was particularly thinking of. It was more just of a 'well, might as well' kinda thing!

@sandmman sandmman merged commit a3dc765 into develop Jan 10, 2018
@tunniclm tunniclm deleted the add_nonarrayget_and_doc branch January 10, 2018 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants