-
Notifications
You must be signed in to change notification settings - Fork 20
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
Enhance Docker Configuration #701
base: dev
Are you sure you want to change the base?
Enhance Docker Configuration #701
Conversation
@@ -4,3 +4,4 @@ distributionUrl=https\://services.gradle.org/distributions/gradle-8.2-all.zip | |||
networkTimeout=10000 | |||
zipStoreBase=GRADLE_USER_HOME | |||
zipStorePath=wrapper/dists | |||
/home/sebas/Android/Sdk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need to remove this...
@@ -84,7 +84,7 @@ module.exports = { | |||
}, | |||
'android.emu.x86_64': { | |||
device: 'emulator_x86_64', | |||
app: 'android.debug' | |||
app: 'Pixel_2_API_30' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need this? the user have to have this specific device in the local env.... mmmm, I don't think so.
@@ -23,6 +23,6 @@ WHO PAYS? | |||
At this time seednode.link is paid for and maintained by operating volunteers. In the future, addresses for contribution to keep the project going may be published. | |||
|
|||
MORE? | |||
Radicle official project chat: https://radicle.zulipchat.com | |||
Radicle official project chat: htt_test_ps://radicle.zulipchat.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this for?
@@ -21,8 +21,7 @@ until [[ $step_complete == true ]]; do | |||
echo "Error: Failed to install APKs" >&2 | |||
echo "For more information see '${output_dir}/emulator_install_apk.txt'" >&2 | |||
exit 1 | |||
fi | |||
fiTHE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this for?
@@ -10,4 +10,5 @@ cd $(git rev-parse --show-toplevel) | |||
|
|||
./scripts/flow_emulator_setup.sh $@ | |||
|
|||
./scripts/flow_emulate_interactive.sh | |||
./scripts/flow_emulate_interactive.sh "start": "react-native start", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see the point of this.... some copy-paste mistake, maybe...
@@ -45,6 +45,7 @@ | |||
"patch-package": "^6.5.1", | |||
"postinstall-postinstall": "^2.1.0", | |||
"prop-types": "^15.8.1", | |||
"punycode": "^2.3.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where or Why are you using this library?
Summary of Changes:
Modified .detoxrc.js: Adjusted emulator settings.
Added Dockerfile: Introduced a Dockerfile for environment setup.
Added .dockerignore: Prevented unnecessary files from being included in Docker images.
Updated gradle.properties: Enabled parallel execution for improved build performance.