Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

Conversation

hyochan
Copy link
Contributor

@hyochan hyochan commented Dec 18, 2018

According to [email protected] default gradle version is now 4.4.

@yuri-kulikov
Copy link
Contributor

Hello @hyochan,
Thanks for contributing!

I'm afraid we still can't merge it. We already had such PR but we decided to wait (while 'compile' works with the latest gradle) to keep backward compatibility as long as possible. Someone, for example, can still use react-native 0.56.1 with the old gradle and the last react-native-code-push version.

@hyochan
Copy link
Contributor Author

hyochan commented Dec 21, 2018

@yuri-kulikov Hi. I think it isn't a good idea because as framework updates, the packages need to update. If you are afraid it isn't backward-compatible, then the users have the choice not to upgrade react-native-code-push version (This could be posted in the readme). In my case, I always have to change this in the local environment because build fails because of this. It isn't forward-compatible (Even when I re-install node_modules). I hoped this to be done for a long time, and I've just created PR because it hasn't been updated for a while.

@yuri-kulikov
Copy link
Contributor

yuri-kulikov commented Dec 21, 2018

@hyochan you're right and we have plans to make it in the nearest future (probably at the beginning of next year). We'll just reopen and merge it when we are ready.

@hyochan
Copy link
Contributor Author

hyochan commented Dec 21, 2018

Thank you.

@hammadzz
Copy link

@yuri-kulikov this is a bit painful that we are holding this in for a few who aren't updating they should need to update for bug fixes. It is now 2019 can we have this in asap?

@yuri-kulikov yuri-kulikov reopened this Jan 15, 2019
Copy link
Contributor

@yuri-kulikov yuri-kulikov left a comment

Choose a reason for hiding this comment

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

We're merging this. Thanks for contributing!

@yuri-kulikov yuri-kulikov merged commit ac51cf6 into microsoft:master Jan 15, 2019
@hyochan hyochan deleted the patch-1 branch February 17, 2019 11:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants