bannzai / SpreadsheetView

Full configurable spreadsheet view user interfaces for iOS applications. With this framework, you can easily create complex layouts like schedule, gantt chart or timetable as if you are using Excel.
MIT License
3.52k stars 458 forks source link

Add CI GitHub Actions #260

Closed bannzai closed 4 years ago

bannzai commented 4 years ago

What

Why

Only testing with travis ci has long time (About 5h...) I try for my idea about decreasing build time for separating CI build and testing each iPhone and cocdcov upload and iPad build.

bannzai commented 4 years ago

Oooops, Xcode 12 will enable after Octobor 20 on GitHub Actions.

https://github.com/actions/virtual-environments/blob/main/images/macos/macos-10.15-Readme.md I will try iOS=13.7

Failure build: https://github.com/bannzai/SpreadsheetView/pull/260/checks?check_run_id=1234720089


xcodebuild: error: Unable to find a destination matching the provided destination specifier:
        { OS:14.0, name:iPhone 11 Pro }

    Unsupported device specifier option.
    The device “My Mac” does not support the following options: name
    Please supply only supported device specifier options.

    Available destinations for the "SpreadsheetView" scheme:
        { platform:macOS, arch:x86_64, variant:Mac Catalyst, id:4203018E-580F-C1B5-9525-B745CECA79EB }
        { platform:iOS Simulator, id:D45E3BEC-7F8A-43E8-90F6-81F7E88834FF, OS:13.7, name:iPad (7th generation) }
        { platform:iOS Simulator, id:F15D83E6-3281-42F5-8675-91A22BD64475, OS:13.7, name:iPad Air (3rd generation) }
        { platform:iOS Simulator, id:3EDB6E03-31BE-4E8B-8E7E-C9E55A432CE5, OS:13.7, name:iPad Pro (9.7-inch) }
        { platform:iOS Simulator, id:B4AAF66C-0EEE-4AD0-B1FD-8FBF3B8CBAF1, OS:13.7, name:iPad Pro (11-inch) (2nd generation) }
        { platform:iOS Simulator, id:8D85B887-1291-43FA-898E-A1495FF41B8D, OS:13.7, name:iPad Pro (12.9-inch) (4th generation) }
        { platform:iOS Simulator, id:9EC32F11-C32B-4900-81A4-F69E927F4433, OS:13.7, name:iPhone 8 }
        { platform:iOS Simulator, id:072D4B47-1E42-4DF7-BE7A-6C77AE11D114, OS:13.7, name:iPhone 8 Plus }
        { platform:iOS Simulator, id:B67C2873-CE1E-40C6-8678-F8B6973A9A19, OS:13.7, name:iPhone 11 }
        { platform:iOS Simulator, id:D03BAA54-7E0F-47FC-8B35-2DB514911998, OS:13.7, name:iPhone 11 Pro }
        { platform:iOS Simulator, id:000E7947-780D-4BB1-B697-6DCF1A243BEC, OS:13.7, name:iPhone 11 Pro Max }
        { platform:iOS Simulator, id:C843081E-65A0-4276-B2EB-FF8D414FBA21, OS:13.7, name:iPhone SE (2nd generation) }

    Ineligible destinations for the "SpreadsheetView" scheme:
        { platform:iOS, id:dvtdevice-DVTiPhonePlaceholder-iphoneos:placeholder, name:Generic iOS Device }
        { platform:iOS Simulator, id:dvtdevice-DVTiOSDeviceSimulatorPlaceholder-iphonesimulator:placeholder, name:Generic iOS Simulator Device }
Error: Process completed with exit code 70.
codecov-io commented 4 years ago

Codecov Report

Merging #260 into master will increase coverage by 17.21%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #260       +/-   ##
===========================================
+ Coverage   76.72%   93.93%   +17.21%     
===========================================
  Files          23        8       -15     
  Lines        1667      132     -1535     
===========================================
- Hits         1279      124     -1155     
+ Misses        388        8      -380     
Impacted Files Coverage Δ
Framework/Sources/Borders.swift 0.00% <ø> (-10.53%) :arrow_down:
Framework/Sources/Cell.swift 100.00% <ø> (+20.00%) :arrow_up:
Framework/Sources/CircularScrolling.swift 98.27% <ø> (+4.48%) :arrow_up:
Framework/Sources/Gridlines.swift 63.63% <ø> (+33.63%) :arrow_up:
Framework/Sources/SpreadsheetView.swift 100.00% <ø> (+15.00%) :arrow_up:
Framework/Sources/Array+BinarySearch.swift
Framework/Sources/SpreadsheetViewDelegate.swift
...ework/Sources/SpreadsheetView+UISnapshotting.swift
Framework/Sources/ScrollPosition.swift
... and 9 more

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 ec707fa...1012671. Read the comment docs.

bannzai commented 4 years ago

Codecovも復活したな。よし

bannzai commented 4 years ago

Successfully spped up testing time on CI. it is reducing 3 hours :tada:

before(4h37min) after(1h39min)