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

Fix breaking change w/ Gradle dependencies #88

Merged
merged 1 commit into from
Aug 1, 2019

Conversation

christophermark
Copy link
Contributor

Intent

This fixes a breaking API change with the Android Gradle plugin, which was deprecated as of version 3.0.0 and later made obsolete.

Changes

When adding Gradle dependencies, the compile command has been removed and replaced with implementation/api.

More information

This fixes a breaking API change with the Android Gradle plugin, which was deprecated as of version 3.0.0 and later made obsolete.

When adding Gradle dependencies, the `compile` command has been removed and replaced with `implementation`/`api`.

More information:
https://docs.gradle.org/5.4.1/userguide/java_library_plugin.html#sec:java_library_separation
https://developer.android.com/studio/build/dependencies?utm_source=android-studio#dependency_configurations
@mvayngrib mvayngrib merged commit b34d764 into tradle:master Aug 1, 2019
@mvayngrib
Copy link
Member

@christophermark thanks!

@christophermark christophermark deleted the bugfix/gradle-api-fixes branch August 7, 2019 15:50
DomiR added a commit to DomiR/react-native-udp that referenced this pull request Nov 22, 2019
* tradle-master:
  Update react-native-udp.podspec
  Use address instead of host name
  feat(socket-lib): udpate GCDAsyncUdpSocket
  Added handling of GCDAsyncUdpSocket's socketDidClose callback in order to bubble notification that the socket has been closed
  Fix breaking change w/ Gradle dependencies (tradle#88)
  2.6.1
  add .git to .npmignore
  2.6.0
  fix max MTU (tradle#84)
  2.5.2
  Update react-native-udp.podspec (tradle#83)
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