Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Cast set_location arguments to float#332

Merged
mykola-mokhnach merged 2 commits intoappium:masterappium/python-client:masterfrom
mykola-mokhnach:locationmykola-mokhnach/python-client:locationCopy head branch name to clipboard
Feb 10, 2019
Merged

Cast set_location arguments to float#332
mykola-mokhnach merged 2 commits intoappium:masterappium/python-client:masterfrom
mykola-mokhnach:locationmykola-mokhnach/python-client:locationCopy head branch name to clipboard

Conversation

@mykola-mokhnach
Copy link
Contributor

Addresses #331

Copy link
Member

@KazuCocoa KazuCocoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

autopep8 exception happened, otherwise LGTM. thanks!

@mykola-mokhnach mykola-mokhnach merged commit 32e46a4 into appium:master Feb 10, 2019
@nunessf
Copy link

nunessf commented Feb 11, 2019

Probably not the place to ask this, but when there might be a release including this update?

@KazuCocoa
Copy link
Member

Just published the latest master as 0.37.
Please check it.

@nunessf
Copy link

nunessf commented Feb 11, 2019

I think it's missing an update on version.py file, because this exception is being thrown:

IOError: [Errno 2] No such file or directory: 'appium\version.py'

@KazuCocoa
Copy link
Member

0.37 package had below python file.mm let me tweak related code.

image

@KazuCocoa
Copy link
Member

I hope this works on your environment...

Below was my env downloaded the module from https://pypi.org/project/Appium-Python-Client/0.38/#files

>>> import appium.common.helper as h
>>> h.library_version()
'0.38'
``

@nunessf
Copy link

nunessf commented Feb 11, 2019

Alright, it's working now, I'm able to use set_location. Thank you!

@mykola-mokhnach mykola-mokhnach deleted the location branch July 18, 2019 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.