-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[Yoga] Increment Yoga version to current, 1.5.0. #91
Conversation
Everything seems to build fine with this version, so I'll seek clarifying information from the reporter of this bug: #25
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.
@appleguy Hey - I'm fine if you merge it whenever you would like to do.
@appleguy Do you use 1.3 already? Should we just merge it for now? |
We are currently on 1.5.0 -- check out the podspec in my latest WIP PR.
Feel free to merge this, does it actually help that other issue? Also fine
to use 1.5.0.
…On Mon, May 15, 2017 at 9:50 AM Michael Schneider ***@***.***> wrote:
@appleguy <https://github.com/appleguy> Do you use 1.3 already? Should we
just merge it for now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#91 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAigAwaAqqwsz-nMAOp-PJ1veEwpoqk1ks5r6IJAgaJpZM4NMe_8>
.
|
@appleguy Let's update this PR to 1.5 than and just merge it. Or what do you think? |
Sounds great!
…On Mon, May 15, 2017 at 10:00 AM Michael Schneider ***@***.***> wrote:
@appleguy <https://github.com/appleguy> Let's update this PR to 1.5 than
and just merge it. Or what do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#91 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAigAyHuc8HsBJ8w7Kk4wEjVYyI9TT2Pks5r6ISYgaJpZM4NMe_8>
.
|
@appleguy Ok cool will do that now. |
@appleguy I will merge this now as it seems the upgrade will work fine. |
This did not fix the issue I had. I have made this PR to show what was needed to make our particular project compile #306 |
* [Yoga] Increment Yoga version to current, 1.3.0 Everything seems to build fine with this version, so I'll seek clarifying information from the reporter of this bug: TextureGroup#25 * Update Yoga version to 1.5.0
Everything seems to build fine with this version, so I'll
seek clarifying information from the reporter of this bug:
#25