Kitura / Kitura-redis

Swift Redis library
Apache License 2.0
94 stars 25 forks source link

Updated to Swift 4.0 #61

Closed Davidde94 closed 6 years ago

Davidde94 commented 6 years ago

The package as a whole has been upgraded to have full support for Swift 4 (with all errors/warnings handled).

Description

Updated code to Swift 4 format (e.g. not using String.characters anymore)

Motivation and Context

This was a requirement in order for https://github.com/IBM-Swift/Kitura-Session-Redis/pull/34 to be merged

How Has This Been Tested?

The pre-existing test suite was run, and all tests passed.

Checklist:

codecov-io commented 6 years ago

Codecov Report

Merging #61 into master will increase coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
+ Coverage   80.93%   80.94%   +0.01%     
==========================================
  Files          17       17              
  Lines        2381     2383       +2     
==========================================
+ Hits         1927     1929       +2     
  Misses        454      454
Flag Coverage Δ
#SwiftRedis 80.94% <100%> (+0.01%) :arrow_up:
Impacted Files Coverage Δ
Sources/SwiftRedis/RedisInfo.swift 91.89% <100%> (+0.46%) :arrow_up:

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 d6626c0...1d7131e. Read the comment docs.