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 android java doc task #450

Merged
merged 1 commit into from
Jan 26, 2022
Merged

Fix android java doc task #450

merged 1 commit into from
Jan 26, 2022

Conversation

poovamraj
Copy link
Contributor

Changes

Please describe both what is changing and why this is important. Include:

androidJavadoc task which isn't used in the project blocked users from integrating the SDK for Gradle 7. We have fixed this by replacing deprecated functionality

References

https://docs.gradle.org/current/userguide/java_plugin.html#tab:configurations

Testing

The SDK has been tested in the latest React Native and Gradle versions as well as the older ones ( RN 0.66.4 & Gradle 6~)

Checklist

@poovamraj poovamraj requested a review from a team as a code owner January 25, 2022 13:22
@poovamraj poovamraj merged commit 2adcbf0 into master Jan 26, 2022
piaskowyk pushed a commit to software-mansion/react-native-reanimated that referenced this pull request Apr 1, 2022
…ity with Gradle 7 (#3120)

## Description

<!--
Description and motivation for this PR.

Inlude Fixes #<number> if this is fixing some issue.

Fixes # .
-->

Fixes #3119. 

## Changes

Use `implementation` instead of `compile`. This is the same fix as auth0/react-native-auth0#450.

<!--
Please describe things you've changed here, make a **high level** overview, if change is simple you can omit this section.

For example:

- Added `foo` method which add bouncing animation
- Updated `about.md` docs
- Added caching in CI builds

-->

<!--

## Screenshots / GIFs

Here you can add screenshots / GIFs documenting your change.

You can add before / after section if you're changing some behavior.

### Before

### After

-->

## Test code and steps to reproduce

Include `react-native-reanimated` with this change in a project using Gradle 7 and it should successfully build.

<!--
Please include code that can be used to test this change and short description how this example should work.
This snippet should be as minimal as possible and ready to be pasted into editor (don't exclude exports or remove "not important" parts of reproduction example)
-->
@evansims evansims deleted the gradle-7-support branch July 5, 2022 20:50
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