Skip to content

Conversation

@bcylin
Copy link
Collaborator

@bcylin bcylin commented Aug 10, 2019

Part the subsequent updates of #9. Update the codebase to the latest Swift syntax and Xcode 10.3.

⬜️ Remove R.swift
⬜️ Replace Freddy with Swift Codable
⬜️ Replace Quick and Nimble with XCTest
✅ Update the project to Xcode 10.3 with Swift 5

⚠️ Not ready to merge until the previous PRs are merged. The base branch needs to be updated.

@bcylin bcylin requested a review from dlackty August 10, 2019 21:41
pod "Hue", "~> 2.0.0"
pod "Kingfisher", "~> 3.2.0"
pod "Hue", git: "https://github.com/zenangst/Hue.git", tag: "5.0.0"
pod "Kingfisher", git: "https://github.com/onevcat/Kingfisher.git", tag: "5.7.0"

Choose a reason for hiding this comment

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

Metrics/LineLength: Line is too long. [82/80]


target :iCookTV do
pod "Alamofire", "~> 4.2.0"
pod "Alamofire", git: "https://github.com/Alamofire/Alamofire.git", tag: "4.8.2"

Choose a reason for hiding this comment

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

Metrics/LineLength: Line is too long. [82/80]

@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #13 into feature/remove-rswift will decrease coverage by 14.82%.
The diff coverage is 21.42%.

Impacted Files Coverage Δ
...CookTV/Protocols/LoadingIndicatorPresentable.swift 0% <0%> (ø) ⬆️
iCookTV/Controllers/VideosViewController.swift 0% <0%> (ø) ⬆️
iCookTV/Views/CategoryCell.swift 0% <0%> (-76.83%) ⬇️
iCookTV/Controllers/VideoPlayerController.swift 0% <0%> (ø) ⬆️
iCookTV/Extensions/Video+PlayerItem.swift 0% <0%> (ø) ⬆️
iCookTV/Helpers/HistoryManager.swift 0% <0%> (ø) ⬆️
iCookTV/Views/MenuButton.swift 0% <0%> (-75%) ⬇️
iCookTV/Protocols/OverlayViewPresentable.swift 0% <0%> (ø) ⬆️
iCookTV/Models/VideosDataSource.swift 0% <0%> (ø) ⬆️
iCookTV/Protocols/VideosGridLayout.swift 0% <0%> (ø) ⬆️
... and 37 more

@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #13 into develop will increase coverage by 10.07%.
The diff coverage is 21.42%.

Impacted Files Coverage Δ
...CookTV/Protocols/LoadingIndicatorPresentable.swift 0% <0%> (ø) ⬆️
iCookTV/Controllers/VideosViewController.swift 0% <0%> (ø) ⬆️
iCookTV/Views/CategoryCell.swift 0% <0%> (ø) ⬆️
iCookTV/Controllers/VideoPlayerController.swift 0% <0%> (ø) ⬆️
iCookTV/Extensions/Video+PlayerItem.swift 0% <0%> (ø) ⬆️
iCookTV/Helpers/HistoryManager.swift 0% <0%> (ø) ⬆️
iCookTV/Views/MenuButton.swift 0% <0%> (ø) ⬆️
iCookTV/Protocols/OverlayViewPresentable.swift 0% <0%> (ø) ⬆️
iCookTV/Models/VideosDataSource.swift 0% <0%> (ø) ⬆️
iCookTV/Protocols/VideosGridLayout.swift 0% <0%> (ø) ⬆️
... and 28 more

@bcylin bcylin changed the base branch from feature/remove-rswift to feature/xctest August 13, 2019 22:33
@bcylin bcylin marked this pull request as ready for review August 13, 2019 22:33
@bcylin bcylin changed the base branch from feature/xctest to develop August 26, 2019 22:51
@polydicebot
Copy link

1 Error
🚫 Please rebase to get rid of the merge commits in this PR

Generated by 🚫 Danger

@dlackty
Copy link
Member

dlackty commented Jan 21, 2020

@bcylin sorry for delaying so long. Could you please help rebase it and I'm gonna merge it then wrap up a new version.

@dlackty dlackty merged commit b721beb into develop Jan 27, 2020
@dlackty dlackty deleted the feature/swift-5 branch January 27, 2020 16:49
@dlackty
Copy link
Member

dlackty commented Jan 27, 2020

@bcylin Thanks! Much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants