-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Android] allow config extension #884
Conversation
@matteo-hertel, thanks for submitting the PR and sorry for the delay. Actually, I think you'll have to wait even a bit more, till @rotemmiz returns from his vacation, that's in a few days. Don't consider the next thing as a request, but I'd be happy to learn (Gradle is not my thing yet, still have to grasp it) - what is the use case that is addressed by this inheritance - does it solve a version conflict in your Gradle builds, speeds them up or anything else? |
Hey @noomorph thanks for your reply The aim here is to have the android configurations in one place (the root project) and have the Detox build file using those config if provided, I have an example of a project that uses Detox in which the It's also standard for android projects to allow this kind of extension, so win win 😄 |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Not stale. |
@rotemmiz, you're more familiar with this area, please tell if this change benefits some project configurations. I don't see proofs like before the change VS after the change (e.g. project fails to build -> project builds successfully), but maybe it's just me and the benefit is already obvious for any person who's good with Gradle. 😕 Techincally, I have no objections regarding the merge as the build passes on Gradle 4.10. |
First, thanks for this PR and sorry for the super long response time. The other part of the solution can come from a different approach, and that is consuming detox from a binary instead of from sources. There's an open PR made by the members of the team, adding aar artifacts to Detox's npm package. This way, you won't have to go through a (potentially) tedious compilation process and adaptations in your project. |
Hi @rotemmiz thanks for getting back to me, my use case was covering the use case of Detox being the only library compiled from source, if the open PR that adds binary to npm covers this use case as well then is a non issue anymore, feel free to close this PR I was happy to contribute to the project 😄 |
👋 Wix!
First of all thanks for making detox, is really amazing 😄
As per the title, I've created this super small PR to allow the android build file to inherit config from the root project, that comes in really handy if the project pulling in detox needs different configs.
I've
stoleborrowed the names from https://github.com/react-native-community/react-native-linear-gradient/blob/master/android/build.gradle given it's from the react native community it should be okI didn't open an issue for this, do you need me to?