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
global: import models from reana-commons #58
Conversation
8f5a854
to
aee0856
Compare
setup.py
Outdated
@@ -72,6 +72,7 @@ | |||
'adage==0.8.5', | |||
'packtivity==0.10.0', | |||
'yadage==0.13.5', | |||
'psycopg2-binary>=2.6.1', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the PostgreSQL dependencies will be coming indirectly via reana-commons
... So perhaps you can already start to proactively declare reana-commons
as a dependency in setup.py
, and we would just modify Dockerfile
to pip-install it from github via requirements-dev.txt
until the times reana-commons
is released on PyPI... Seems more forward-friendly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes exactly, and that's a good idea, I'll go with this
@@ -29,15 +29,15 @@ | |||
from yadage.steering_api import steering_ctx | |||
from yadage.utils import setupbackend_fromstring | |||
|
|||
from reana_commons.models import Workflow, WorkflowStatus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please invert the two lines to make isort
in Travis CI happy
da8f5df
to
f46778a
Compare
@dinosk can you have a look look to the latest changes? |
4bb0f0a
to
b32dc12
Compare
b32dc12
to
ed0455c
Compare
Signed-off-by: Dinos Kousidis <dinos.kousidis@cern.ch>
ed0455c
to
a8821bc
Compare
Signed-off-by: Dinos Kousidis dinos.kousidis@cern.ch