-
Notifications
You must be signed in to change notification settings - Fork 363
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
added ability to override capabilites from environment variables #473
added ability to override capabilites from environment variables #473
Conversation
@anandbagmar How would you pass the app capability for both iOS real device and iOS Simulator? |
@saikrishna321 - if we want Platform specific updates, I can update the PR based on that. Would that approach be better? |
@anandbagmar We need to change approach after this PR got merged #478 Now we will default to "app": { |
Cool. Will take a look and update the PR |
@anandbagmar This will be very helpful. I'm also stuck at this point. I have to use APK path from Bitrise CI. Would be really nice if you finish this |
pull from ATD to fork
…ome property, we need to pass this as an environment variable in a specific format: ex: to update android's app path, set env var as: 'atd.android.app' ex: To update a specific property as part of an array, 'atd.hostMachines.0.appiumServerPath=.... ' ex: To update the 1st simulator's device name in the 1st hostMachine: 'atd.hostMachines.0.simulators.0.deviceName
@saikrishna321 , @SrinivasanTarget - I have updated this PR. Here are some examples how to update the capabilities using environment variables:
Let me know if this seems ok |
@saikrishna321 , @SrinivasanTarget - thanks for merging the PR. Any idea when will this be available via maven, and with what version #? |
The next version will be major release v12.0.0 - cloud support and contract change on app capability. Once the docs are done we can release |
ok. Hopefully that happens soon :) Thanks! |
added ability to override capabilites from environment variables