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

Limit source file types to Swift and headers #38

Merged
merged 2 commits into from
Aug 25, 2017
Merged

Limit source file types to Swift and headers #38

merged 2 commits into from
Aug 25, 2017

Conversation

mergesort
Copy link
Contributor

This fixes a bug where the Info.plist file from this repo was included in the compile sources phase, which causes a build failure with the new Xcode 9 build system.

This fixes a bug where the Info.plist file from this repo was included in the compile sources phase, which causes a build failure with the new Xcode 9 build system.
@@ -12,6 +12,6 @@ StackViewController is a Swift framework that simplifies the process of building
s.author = { "Indragie Karunaratne" => "i@indragie.com" }
s.source = { :git => "https://github.com/seedco/StackViewController.git", :tag => s.version.to_s }
s.ios.deployment_target = '9.0'
s.source_files = 'StackViewController/**'
s.source_files = 'StackViewController/**/*.{h,swift}'
Copy link
Contributor

Choose a reason for hiding this comment

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

@mergesort thank you for the PR!
Did you mean to have source files set to StackViewController/*.{h,swift} ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, my mistake. Lemme tweak that.

@mergesort
Copy link
Contributor Author

Updated with the right file path.

@vikmeup
Copy link
Contributor

vikmeup commented Aug 25, 2017

@mergesort 👍

@vikmeup vikmeup merged commit 5fba46b into seedco:master Aug 25, 2017
@mergesort
Copy link
Contributor Author

@vikmeup I just realized that I didn't bump the version number. Would it make sense to give it a version number of 0.3.1 in another patch? I don't have CocoaPods push access, so it would make sense to come from your side if you do choose to do that.

Thanks!

@vikmeup
Copy link
Contributor

vikmeup commented Aug 28, 2017

@mergesort I had problems running pod trunk push, I can take a look little later to push cocoa pods update, at the main time can you point to master? thanks!

@vikmeup
Copy link
Contributor

vikmeup commented Aug 28, 2017

@mergesort done! version 0.3.1 successfully published.

@mergesort
Copy link
Contributor Author

Thanks so much!

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