Skip to content

add buildconfig flag for AGP8+#2599

Merged
m-bert merged 2 commits into
software-mansion:mainfrom
pvegh:fix-agp8-buildConfig
Sep 27, 2023
Merged

add buildconfig flag for AGP8+#2599
m-bert merged 2 commits into
software-mansion:mainfrom
pvegh:fix-agp8-buildConfig

Conversation

@pvegh

@pvegh pvegh commented Sep 21, 2023

Copy link
Copy Markdown
Contributor

Description

Add support for AGP8 by adding the buildConfig Flag indicating that this library uses custom buildconfig fields and needs it's BuildConfig class generated.
Fixes #2571

Test plan

No idea, need some help here since I'm not a RN person. I'm just trying to push support for AGP8 so we don't have to enable generating BuildConfig classes generally in our project, which has a performance impact.

@m-bert m-bert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hi @pvegh! Sorry for late response.

This PR looks fine, could you please merge main branch and add to description that it fixes #2571? Then we can merge it.

@pvegh

pvegh commented Sep 27, 2023

Copy link
Copy Markdown
Contributor Author

Done

@m-bert m-bert merged commit bb57a8f into software-mansion:main Sep 27, 2023
@pvegh pvegh deleted the fix-agp8-buildConfig branch September 27, 2023 12:57
renovate Bot referenced this pull request in valora-xyz/wallet Oct 15, 2023
…4315)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[react-native-gesture-handler](https://togithub.com/software-mansion/react-native-gesture-handler)
| [`^2.13.1` ->
`^2.13.2`](https://renovatebot.com/diffs/npm/react-native-gesture-handler/2.13.1/2.13.2)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/react-native-gesture-handler/2.13.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/react-native-gesture-handler/2.13.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/react-native-gesture-handler/2.13.1/2.13.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/react-native-gesture-handler/2.13.1/2.13.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>software-mansion/react-native-gesture-handler
(react-native-gesture-handler)</summary>

###
[`v2.13.2`](https://togithub.com/software-mansion/react-native-gesture-handler/releases/tag/2.13.2)

[Compare
Source](https://togithub.com/software-mansion/react-native-gesture-handler/compare/2.13.1...2.13.2)

#### 🐛 Bug fixes

- Add `ghQueueMicrotask` by
[@&#8203;m-bert](https://togithub.com/m-bert) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2612](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2612)

#### 👍 Improvements

- New onSwipeableOpenDragStart and onSwipeableCloseDragStart events by
[@&#8203;rampazzo1989](https://togithub.com/rampazzo1989) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2589](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2589)
- Remove jcenter by
[@&#8203;ChaseKnowlden](https://togithub.com/ChaseKnowlden) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2610](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2610)
- Add buildconfig flag for AGP8+ by
[@&#8203;pvegh](https://togithub.com/pvegh) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2599](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2599)

#### 🔢 Miscellaneous

- Update dependencies in e2e/web-tests by
[@&#8203;m-bert](https://togithub.com/m-bert) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2601](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2601)
- Bump [@&#8203;sideway/formula](https://togithub.com/sideway/formula)
from 3.0.0 to 3.0.1 in /FabricExample by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2605](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2605)
- Bump [@&#8203;sideway/formula](https://togithub.com/sideway/formula)
from 3.0.0 to 3.0.1 in /docs by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2602](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2602)
- Give some love to Gesture Handler docs by
[@&#8203;kacperkapusciak](https://togithub.com/kacperkapusciak) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2607](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2607)
- Fix typo in Composing gestures docs by
[@&#8203;S1D007](https://togithub.com/S1D007) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2614](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2614)
- Bump postcss from 8.4.30 to 8.4.31 in /e2e/web-tests by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2623](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2623)
- Bump postcss from 8.4.14 to 8.4.31 in /docs by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2621](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2621)
- Bump [@&#8203;sideway/formula](https://togithub.com/sideway/formula)
from 3.0.0 to 3.0.1 in /example by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2606](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2606)

#### New Contributors

- [@&#8203;rampazzo1989](https://togithub.com/rampazzo1989) made their
first contribution in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2589](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2589)
- [@&#8203;ChaseKnowlden](https://togithub.com/ChaseKnowlden) made their
first contribution in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2610](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2610)
- [@&#8203;pvegh](https://togithub.com/pvegh) made their first
contribution in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2599](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2599)
- [@&#8203;S1D007](https://togithub.com/S1D007) made their first
contribution in
[https://github.com/software-mansion/react-native-gesture-handler/pull/2614](https://togithub.com/software-mansion/react-native-gesture-handler/pull/2614)

**Full Changelog**:
software-mansion/react-native-gesture-handler@2.13.1...2.13.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 5pm,every weekend" in timezone
America/Los_Angeles, Automerge - "after 5pm,every weekend" in timezone
America/Los_Angeles.

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/valora-inc/wallet).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy44LjEiLCJ1cGRhdGVkSW5WZXIiOiIzNy44LjEiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: valora-bot <valorabot@valoraapp.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.

Add AGP8 support by adding buildConfig = true to buildFeatures

2 participants