Skip to content

Conversation

@stefanomondino
Copy link
Member

@stefanomondino stefanomondino commented Sep 18, 2018

I've also already updated the Podspec to 3.7 for future cocoapods release

Fixes #150. Fixes #152.

PRODUCT_NAME = "$(TARGET_NAME)";
SWIFT_ACTIVE_COMPILATION_CONDITIONS = DEBUG;
SWIFT_OPTIMIZATION_LEVEL = "-Onone";
SWIFT_SWIFT3_OBJC_INFERENCE = On;
Copy link
Contributor

Choose a reason for hiding this comment

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

I think @objc inference should be set to default.

PRODUCT_BUNDLE_IDENTIFIER = com.ashfurrow.Action;
PRODUCT_NAME = "$(TARGET_NAME)";
SKIP_INSTALL = YES;
SWIFT_SWIFT3_OBJC_INFERENCE = On;
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here.

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

Copy link
Member

@ashfurrow ashfurrow left a comment

Choose a reason for hiding this comment

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

Looks great, pending answering the question from @mAu888 – thanks!

@stefanomondino
Copy link
Member Author

Should be ok now. thank you for the review :)

@ashfurrow ashfurrow merged commit 8957f38 into master Sep 18, 2018
@ashfurrow ashfurrow deleted the swift_4.2 branch September 18, 2018 16:38
@ashfurrow
Copy link
Member

Cool, thanks again! I'll get this released to CocoaPods trunk and tag a release on GitHub right away 👍

@ashfurrow
Copy link
Member

Oh boo, I don't have an adequate Xcode version. Let me download it...

@ashfurrow
Copy link
Member

Okay all done! Thanks again 👍

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.

4 participants