Deprecated youiengine#652
Deprecated youiengine#652TikhomirovSergey merged 3 commits intoappium:masterappium/java-client:masterfrom TikhomirovSergey:deprecated_youiengineCopy head branch name to clipboard
Conversation
|
@SrinivasanTarget @mykola-mokhnach Could you review it. FYI @pfoster-youitv @philgreenyouilabs @jlipps @simongranger |
|
|
||
| import java.net.URL; | ||
|
|
||
| @Deprecated |
There was a problem hiding this comment.
I suppose the common practice is to put a docsting before an annotation, but not after
|
@mykola-mokhnach @SrinivasanTarget I have updated it a little bit and now I am trying to notify youiengine guys. |
|
LGTM @TikhomirovSergey |
|
@TikhomirovSergey I think the best place to find them is general channel of Appium's slack. We could add your account if you are still not there. |
That was the main thing we needed and not be strictly tied to either iOS or Android elements/commands since we do not use Android or iOS native elements. 👍 |
|
@pfoster-youitv |
Change list
io.appium.java_client.youienginepackage was marked asDeprecatedcom.google.code.gson:gsonto v2.8.1Types of changes
Details
This package was added at #429. It was accepted due to previous implementation was not flexible and it did not allowed user to use their own commands/JSONWP protocol extensions. Now
AppiumDriveris not abstract and it can be used across supported platfoms.Please take a look at the following sample:
Also now it allows end user to extend command repository the following way:
Also I was motivated to remove this package because there is no activity/development.