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

As of todays update, I am running into this issue. 'common' renamed RunLoopMode.commonModes #3647

Closed
sdevo619 opened this issue Sep 18, 2018 · 9 comments

Comments

@sdevo619
Copy link

What did you do?

Updated to new version of Charts on an iOS 12 project by running pod update in terminal

What did you expect to happen?

no errorrs

What happened instead?

screen shot 2018-09-17 at 7 34 57 pm

screen shot 2018-09-17 at 7 42 45 pm

Charts version/Branch/Commit Number:
Xcode version: 10
Swift version: 4.1
Platform(s) running Charts: 12
macOS version running Xcode:

Demo Project

ℹ Please link to or upload a project we can download that reproduces the issue.

@muzoman
Copy link

muzoman commented Sep 18, 2018

Me too

@CoderDoraemon
Copy link

CoderDoraemon commented Sep 18, 2018

在pod 添加 试试

post_install do |installer|

  installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
          if target.name == 'Charts'
              config.build_settings['SWIFT_VERSION'] = '4.2'
              else
              config.build_settings['SWIFT_VERSION'] = '4.1'
          end
      end
  end

end

@muzoman
Copy link

muzoman commented Sep 18, 2018

在pod 添加

post_install do |installer|

  installer.pods_project.targets.each do |target|
      target.build_configurations.each do |config|
          if target.name == 'Charts'
              config.build_settings['SWIFT_VERSION'] = '4.2'
              else
              config.build_settings['SWIFT_VERSION'] = '4.1'
          end
      end
  end

end

Where is this code placed?

@CoderDoraemon
Copy link

image

@cythb
Copy link

cythb commented Sep 19, 2018

Please try this version:https://github.com/danielgindi/Charts/tree/4.0.0

@liuxuan30
Copy link
Member

@cythb stop asking people to try 4.0 branch. IT'S NOT FOR PUBLIC. Next time I saw you post like this, I will delete it.

@liuxuan30
Copy link
Member

liuxuan30 commented Sep 19, 2018

Please note, we try to make sure master branch always work with latest Xcode and Swift; if you plan to upgrade your EVERY swift library to 4.2, you can take the new code. Otherwise, stick to old Chart library.

The new commits for Swift 4.2 will not work with lower versions.

@SURYAKANTSHARMA
Copy link

SURYAKANTSHARMA commented Sep 24, 2018

@srn214 , @muzoman I had encounter same issue, thanks for solution 👍 , Could you please explain a bit more. Is this is setting swift version in build setting to 4.2 but why its working. Any help appreciated.

@Jovisx
Copy link

Jovisx commented Dec 30, 2018

i just correct the code with

forMode: RunLoop.Mode.common

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

No branches or pull requests

7 participants