Tset-Noitamotua / Sikuli-and-Robot-Framework-Integration

Windows 7 / 8.1 + Sikuli 1.1.0 compatible version of source for the great tuturial from Mike´s blog
http://blog.mykhailo.com/2011/02/how-to-sikuli-and-robot-framework.html
10 stars 5 forks source link

importing Sikuli stuff #2

Closed Tset-Noitamotua closed 9 years ago

Tset-Noitamotua commented 9 years ago

For calc.py in step_1 would it be sufficient and ok to import this?

from sikuli import JEnv
from sikuli import Env
from sikuli import setBundlePath
from sikuli.Sikuli import App
from sikuli import Region
from sikuli import Screen
from sikuli import KEY_CTRL

Instead of doing this:

from sikuli import *
RaiMan commented 9 years ago

No, please do not do that.

The wrapper sikuli.Sikuli.py is there to separate the Jython from the Java level and assures, that works, what is mentioned in the docs. It leaves the freedom to me, to change what I think is needed under the hood and still having a layer, that keeps things compatible.

Of course, you are free, to do what you want, but in case of problems, I will insist on using from sikuli import *

and ask you to reproduce the problem.

Tset-Noitamotua commented 9 years ago

Ok. I will consider it in the future.