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

feat: add app_path property ("appPath") to Mac2Options #1014

Merged
merged 2 commits into from
Aug 17, 2024

Conversation

a-fultz
Copy link
Contributor

@a-fultz a-fultz commented Aug 16, 2024

This PR adds support for setting the optional appPath capability via Mac2Options().app_path, aligning with the related "bundle_id" setting. Presently it is necessary to pass in a value for appPath as part of the capabilities dictionary to Mac2Options().load_capabilities() if you wish to set it.

appPath is implemented in the Mac2 driver:
https://github.com/appium/appium-mac2-driver?tab=readme-ov-file#capabilities

@mykola-mokhnach
Copy link
Contributor

Thank you @a-fultz

Could you please rename the PR to satisfy conventional commits naming rules.
Also the CI is not very happy:

ERROR: /home/runner/work/python-client/python-client/appium/options/mac/mac2/app_path_option.py Imports are > incorrectly sorted and/or formatted.
Skipped 2 files
make: *** [Makefile:10: isort] Error 1
Please run command 'make isort' on your local and commit the result

@a-fultz a-fultz changed the title add app_path property ("appPath") to Mac2Options feat: add app_path property ("appPath") to Mac2Options Aug 17, 2024
@KazuCocoa KazuCocoa merged commit 18c4723 into appium:master Aug 17, 2024
16 of 17 checks passed
@KazuCocoa KazuCocoa added the size:S contribution size: S label Sep 6, 2024
@jlipps
Copy link
Member

jlipps commented Sep 6, 2024

Hi @a-fultz , congrats, the Appium project wants to compensate you for this (and perhaps other) contribution(s) this month! Please reply to this comment mentioning me and sharing your OpenCollective account name, so that we can initiate payment! Or let me know if you decline to receive compensation via OpenCollective. Thank you!

@a-fultz
Copy link
Contributor Author

a-fultz commented Sep 6, 2024

@jlipps it is af2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S contribution size: S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants