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

Enabled Examples building on Xcode 11 #619

Closed
wants to merge 1 commit into from

Conversation

iDevid
Copy link
Collaborator

@iDevid iDevid commented Oct 7, 2019

Enabled building for examples with Xcode 11. We should remove Chameleon Pod, as it is no longer maintained.

@@ -4,7 +4,7 @@ target 'HeroExamples' do
platform :ios, '9.0'
use_frameworks!
pod "CollectionKit"
pod 'ChameleonFramework/Swift', :git => 'https://github.com/ViccAlexander/Chameleon.git'
pod 'ChameleonFramework/Swift', :git => 'https://github.com/wowansm/Chameleon.git', :branch => 'swift5'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should wait until mainline is swift5 compatible if not already.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Project is no longer maintened. We should remove it from the examples.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Makes sense to me, wanna just remove it within this PR then?

@JoeMatt JoeMatt changed the base branch from master to develop October 7, 2019 23:30
@JoeMatt JoeMatt added this to the 1.6.0 milestone Oct 29, 2019
@JoeMatt
Copy link
Collaborator

JoeMatt commented Feb 8, 2021

no longer required

@JoeMatt JoeMatt closed this Feb 8, 2021
@unfurl-links unfurl-links bot mentioned this pull request Feb 15, 2021
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.

None yet

2 participants