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

React Native 0.64.X android builds will fail with current build command #100

Closed
kieran-osgood opened this issue May 10, 2021 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@kieran-osgood
Copy link
Contributor

kieran-osgood commented May 10, 2021

Orb version

v5.1.0

What happened

I've been trying to fix some issues with my app running with the orb, I'd opened an issue here which describes it fully, believing it to be related to the tool I used to initialise the app: infinitered/ignite#1676 , but it seems that since react-native 0.64 there has been a bump to the minSdkVersion, see lines 392:

https://github.com/facebook/react-native/blob/0.63-stable/ReactAndroid/build.gradle
https://github.com/facebook/react-native/blob/0.64-stable/ReactAndroid/build.gradle

As a result this causes the assembleAndroidTest to fail, meaning the build fails and can't run detox tests as no APK produced.

Now I'm not exactly certain on the entire process the orb has, so forgive me if this is a naive approach, I was wondering if:

  1. adjusting the command to target just app:assembleAndroidTest
  2. adding the option to remove it from the build job
  3. move it to the test job
    would be suitable?

I was thinking I'd add a PR for an additional string, or enum parameter [assembleAndroidTest,app:assembleAndroidTest, ``] (figure this guides people to either be all the tests, just app, or none, but string would be more flexible)- that allows users to adjust the command for react-native 0.64+ users whilst package maintainers increase their minSdkVersion?

@kieran-osgood
Copy link
Contributor Author

I added a PR: #101 - happy to adjust/discuss options if I'm missing the mark or want it differently?

@matt-oakes
Copy link
Member

Close and fixed in v5.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants