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

Auto add proguard config #3494

Merged
merged 6 commits into from
Feb 15, 2023
Merged

Conversation

heroic
Copy link
Contributor

@heroic heroic commented Aug 18, 2022

Description

This will automatically configure proguard rules. Preventing crahes for people who miss out on adding these steps, or use expo with proguard.

Changes

Added proguard rules config, and enabled consumerProguardFiles in build.gradle.

Test code and steps to reproduce

All tests pass.

Checklist

  • Included code example that can be used to test this change
  • Updated TS types
  • Added TS types tests
  • Added unit / integration tests
  • Updated documentation
  • Ensured that CI passes

@piaskowyk piaskowyk self-requested a review August 19, 2022 11:10
@heroic
Copy link
Contributor Author

heroic commented Aug 19, 2022

@piaskowyk looks like this failed on some non-linked code.

@heroic
Copy link
Contributor Author

heroic commented Aug 26, 2022

@piaskowyk could you help with running the workflows again?

@tomekzaw tomekzaw added this pull request to the merge queue Feb 15, 2023
Merged via the queue into software-mansion:main with commit 18fb8d3 Feb 15, 2023
fluiddot pushed a commit to wordpress-mobile/react-native-reanimated that referenced this pull request Jun 5, 2023
## Description

<!--
Description and motivation for this PR.

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

Fixes # .
-->

This will automatically configure proguard rules. Preventing crahes for
people who miss out on adding these steps, or use expo with proguard.

## Changes

<!--
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

-->

Added proguard rules config, and enabled consumerProguardFiles in
build.gradle.

## Test code and steps to reproduce

All tests pass.

## Checklist

- [ ] Included code example that can be used to test this change
- [ ] Updated TS types
- [ ] Added TS types tests
- [ ] Added unit / integration tests
- [x] Updated documentation
- [x] Ensured that CI passes

---------

Co-authored-by: Krzysztof Piaskowy <krzychu2956@gmail.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.

3 participants