Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update project configuration #9

Merged
merged 6 commits into from
Aug 13, 2019
Merged

Conversation

bcylin
Copy link
Collaborator

@bcylin bcylin commented Aug 10, 2019

This PR separates the changes of project config from #8.

It seems to be very difficult to migrate the project from Swift 3 to Swift 5 at once. One possible approach is to reduce third party dependencies and update different parts of the project separately.

It updates Gems and fixes SwiftLint warnings, getting prepared for the migration to Swift 5. CocoaPods is locked at 1.3.1 with Xcode 9.2 to keep the CI running. #trivial

Subsequent updates:

  • Close Update Gems and remove R.swift #8
  • Remove R.swift
  • Replace Freddy with Swift Codable
  • Replace Quick and Nimble with XCTest
  • Update the project to Xcode 10.3 with Swift 5

@bcylin bcylin requested a review from dlackty August 10, 2019 20:09
if let cell = dequeueReusableCell(withReuseIdentifier: type.reuseIdentifier, for: indexPath) as? T {
return cell
} else {
return type.init()
}
}

func dequeueReusableSupplementaryView<T: UICollectionReusableView>(type: T.Type = T.self, ofKind kind: String, for indexPath: IndexPath) -> T where T: Reusable {
func dequeueReusableSupplementaryView<T: UICollectionReusableView>(type: T.Type = T.self, ofKind kind: String, for indexPath: IndexPath) -> T {

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 120 characters or less: currently 145 characters (line_length)

@polydicebot
Copy link

1 Error
🚫 Please update CHANGELOG.md.

Generated by 🚫 Danger

@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #9 into develop will decrease coverage by 1.06%.
The diff coverage is 35.71%.

Impacted Files Coverage Δ
iCookTV/Controllers/HistoryViewController.swift 0% <0%> (ø) ⬆️
iCookTV/Helpers/HistoryManager.swift 0% <0%> (ø) ⬆️
iCookTV/Protocols/OverlayViewPresentable.swift 0% <0%> (ø) ⬆️
iCookTV/AppDelegate.swift 34.61% <0%> (+0.13%) ⬆️
iCookTV/Extensions/DataRequest+Result.swift 65.21% <100%> (-12.21%) ⬇️
iCookTV/Controllers/LaunchViewController.swift 95.94% <100%> (+0.44%) ⬆️
iCookTV/Protocols/Reusable.swift 43.75% <50%> (+0.27%) ⬆️
iCookTV/Helpers/GroundControl.swift 22.22% <0%> (-16.24%) ⬇️
iCookTV/Protocols/DataFetching.swift 70% <0%> (-7.78%) ⬇️
... and 46 more

1 similar comment
@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #9 into develop will decrease coverage by 1.06%.
The diff coverage is 35.71%.

Impacted Files Coverage Δ
iCookTV/Controllers/HistoryViewController.swift 0% <0%> (ø) ⬆️
iCookTV/Helpers/HistoryManager.swift 0% <0%> (ø) ⬆️
iCookTV/Protocols/OverlayViewPresentable.swift 0% <0%> (ø) ⬆️
iCookTV/AppDelegate.swift 34.61% <0%> (+0.13%) ⬆️
iCookTV/Extensions/DataRequest+Result.swift 65.21% <100%> (-12.21%) ⬇️
iCookTV/Controllers/LaunchViewController.swift 95.94% <100%> (+0.44%) ⬆️
iCookTV/Protocols/Reusable.swift 43.75% <50%> (+0.27%) ⬆️
iCookTV/Helpers/GroundControl.swift 22.22% <0%> (-16.24%) ⬇️
iCookTV/Protocols/DataFetching.swift 70% <0%> (-7.78%) ⬇️
... and 46 more

Copy link
Member

@dlackty dlackty left a comment

Choose a reason for hiding this comment

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

Thanks @bcylin! This is huge and resolved several outdated configs.

@dlackty dlackty mentioned this pull request Aug 13, 2019
5 tasks
@bcylin bcylin merged commit 5bf02d0 into develop Aug 13, 2019
@bcylin bcylin deleted the feature/update-project-config branch August 13, 2019 22:37
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.

4 participants