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 xcode project for xcode 9 #2767

Merged
merged 6 commits into from
Sep 3, 2017
Merged

Update xcode project for xcode 9 #2767

merged 6 commits into from
Sep 3, 2017

Conversation

pmairoldi
Copy link
Collaborator

This update the Charts project to support Xcode 9 and also fix #2593.

@codecov-io
Copy link

codecov-io commented Sep 2, 2017

Codecov Report

Merging #2767 into master will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2767      +/-   ##
==========================================
+ Coverage   19.65%   19.73%   +0.08%     
==========================================
  Files         112      113       +1     
  Lines       13715    13730      +15     
==========================================
+ Hits         2695     2710      +15     
  Misses      11020    11020
Impacted Files Coverage Δ
Tests/Charts/ChartUtilsTests.swift 100% <ø> (ø) ⬆️
Tests/Charts/Snapshot.swift 100% <100%> (ø)
Tests/Charts/LineChartTests.swift 100% <100%> (ø) ⬆️
Tests/Charts/BarChartTests.swift 100% <100%> (ø) ⬆️

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 79b6049...d300d6d. Read the comment docs.

@pmairoldi pmairoldi force-pushed the xcode-9 branch 7 times, most recently from c2b5d9a to 1fdc212 Compare September 3, 2017 19:36
@pmairoldi pmairoldi force-pushed the xcode-9 branch 2 times, most recently from 1025a30 to 8a9dafd Compare September 3, 2017 22:20
@liuxuan30
Copy link
Member

Hi @petester42, this also included #2507, right?

@pmairoldi
Copy link
Collaborator Author

No. Not updated for swift 4. Just making sure project structure is good, code coverage bug is fixed and ci passes.

@liuxuan30
Copy link
Member

ah ok, thanks

PeterSrost pushed a commit to sokol8/Charts that referenced this pull request Oct 31, 2018
Update xcode project to support xcode 9
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.

Invalid Bundle - Xcode 9 Beta 2
3 participants