Skip to content

Clean up converted Swift 3#100

Merged
mattrubin merged 3 commits into
swift-3from
swift-3-cleanup
Nov 12, 2016
Merged

Clean up converted Swift 3#100
mattrubin merged 3 commits into
swift-3from
swift-3-cleanup

Conversation

@mattrubin

Copy link
Copy Markdown
Owner

Clean up some of the converted Swift 3 code.

Comment thread Sources/Token+URL.swift

private func parse<P, T>(_ item: P?, with parser: ((P) -> T?), defaultTo defaultValue: T? = nil,
overrideWith overrideValue: T? = nil) -> T? {
private func parse<P, T>(_ item: P?, with parser: ((P) -> T?), defaultTo defaultValue: T? = nil, overrideWith overrideValue: T? = nil) -> T? {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line Length Violation: Line should be 100 characters or less: currently 142 characters (line_length)

@codecov-io

codecov-io commented Nov 12, 2016

Copy link
Copy Markdown

Current coverage is 91.59% (diff: 100%)

Merging #100 into swift-3 will decrease coverage by 0.04%

@@            swift-3       #100   diff @@
==========================================
  Files             6          6          
  Lines           371        369     -2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits            340        338     -2   
  Misses           31         31          
  Partials          0          0          

Powered by Codecov. Last update 38ac80b...7e2574a

@mattrubin mattrubin merged commit 2b54c6d into swift-3 Nov 12, 2016
@mattrubin mattrubin deleted the swift-3-cleanup branch November 12, 2016 02:09
@mattrubin mattrubin added this to the 3.0 milestone Nov 12, 2016
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