-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Podspec added #24
Podspec added #24
Conversation
Added instructions to Readme |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@stephenkopylov Did you see my comment review?
s.preserve_paths = 'README.md', 'package.json', '*.js' | ||
s.source_files = 'ios/ReactNativeKeyboardManager/**/*.{h,m}' | ||
s.dependency 'React' | ||
s.dependency 'IQKeyboardManager' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s.dependency 'IQKeyboardManager'
This dependency of the IQKeyboardManager
make Cocopods to install it again?
Because the IQKeyboardManager
is already included in the source code.
Yep, I've seen. In case when user uses cocoapods it will be handled by it, otherwise it will use your, included IQKeyboardManager. It's actually a one and only proper way to handle dependencies in cocoapods. UPD: and yep, it works with charm this way =) |
Great, thanks. |
I'm going to release a version in a few hours. |
@douglasjunior |
Added a proper podspec for using with cocoapods.
Actually, this is the only way to link the library with project without suffering and this shitty
"<RCT***.h> not found" errors
Usage:
pod 'RNKeyboardManager', :path => '../node_modules/react-native-keyboard-manager'