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

CocoaPods - Properly Interpolate Tag Version #1174

Merged
merged 1 commit into from
Jan 2, 2020
Merged

Conversation

ptlu
Copy link
Contributor

@ptlu ptlu commented Nov 16, 2019

I faced an issue today where CocoaPods could not properly install this library because the tag version was incorrect. This should fix that!

Why?

  • Ruby uses " quotes when interpolation is required. Using ' will mean v#{version}
    is used directly rather than providing the version number.

Changes:

  1. User Ruby string interpolation when setting the tag version for the source.

Why?
* Ruby uses `"` quotes when interpolation is required.  Using `'` will mean `v#{version}` 
  is used directly rather than providing the version number.

Changes:
1. User Ruby string interpolation when setting the tag version for the source.
@ivpusic ivpusic merged commit 74751d5 into ivpusic:master Jan 2, 2020
@ivpusic
Copy link
Owner

ivpusic commented Jan 2, 2020

thanks

@ptlu ptlu deleted the patch-1 branch January 19, 2020 20:02
ivpusic pushed a commit that referenced this pull request May 1, 2020
Why?
* Ruby uses `"` quotes when interpolation is required.  Using `'` will mean `v#{version}` 
  is used directly rather than providing the version number.

Changes:
1. User Ruby string interpolation when setting the tag version for the source.
ivpusic added a commit that referenced this pull request May 1, 2020
* Switch Android SDK 29 deprecated methods to alternatives (#1110)

Android SDK 29 deprecated the following method for getting external directories
that we use repeatedly.
https://developer.android.com/reference/android/os/Environment#getExternalStoragePublicDirectory(java.lang.String)

I converted the various use cases to use the suggested alternative in
the Context API.
https://developer.android.com/reference/android/content/Context.html#getExternalFilesDir(java.lang.String)

* Updated podspec

* Added iOS 13 DarkMode for picker (#1129)

* Update README.md

* version bump

* Removed isMovingToParentViewController so scroll to bottom works (#1137)

* Fix deprecated RCTImageLoader.h import (#1142)

* upgrade example project to react-native 0.61

* version bump

* Update README.md

* Fix crash after video recording in ios13 & xcode11 (#1145)

* Fix crash after video recording in ios13 & xcode11

UI-related code should run in main thread, as exception says:

`Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'threading violation: expected the main thread'`

* Typo fix

* Fix xCode warnings about main thread execution (#1146)

This kind of warnings also presents in output console if Main Thread Checker is active

* ignore iml files

* version bump

* Fix typo (#1150)

Changed the word "noticable" to "noticeable".

* update type (#1200)

* CocoaPods - Properly Interpolate Tag Version (#1174)

Why?
* Ruby uses `"` quotes when interpolation is required.  Using `'` will mean `v#{version}` 
  is used directly rather than providing the version number.

Changes:
1. User Ruby string interpolation when setting the tag version for the source.

* small typing improvements

* version bump

* fix typings for writeTempFile

* version bump

* (ios) changing images sort order

* upgrade react-native-cli

* using local QBImagePicker as podspec

* Update README.md (#1265)

Clarify that avoidEmptySpaceAroundImage option is (ios only)

* testing with sdk 29

* update yarn.lock

* version bump

Co-authored-by: Phil Simmons <phil@flyclops.com>
Co-authored-by: Yury Korzun <yury@flyclops.com>
Co-authored-by: cr1s3c <yannre1210@gmail.com>
Co-authored-by: Filipe Merker <filipesmerker@gmail.com>
Co-authored-by: Artur Eshenbrener <strate@yandex.ru>
Co-authored-by: Suraneti Rodsuwan <suraneti.rod@gmail.com>
Co-authored-by: mikunimaru <43168745+mikunimaru@users.noreply.github.com>
Co-authored-by: Peter Lucak <login@peterlucak.com>
Co-authored-by: Ivan Pusic <ivpusic@users.noreply.github.com>
Co-authored-by: Mujtaba F. Radhi <al.mujtaba@hotmail.com>
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.

2 participants