-
Notifications
You must be signed in to change notification settings - Fork 125
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
Dockerfile for ROSE #428
Dockerfile for ROSE #428
Conversation
Fixes RedHat-Israel#173 Uses official CentOS Python 2.7 image.
Dockerfile
Outdated
@@ -0,0 +1,26 @@ | |||
# Use official CentOS Python 2.7 image |
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.
@rollandf , shoulds we use python 3.6 for it?
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.
Python 2.7 is not supported, so yes.
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.
Hi, i'm not sure where you see this line.
In the files I commited the dicker uses python 3.7
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.
Maybe after rebasing it will be removed.
You can see another reference at line 3.
## Running a driver | ||
|
||
In a new window, open your virtual environment: | ||
|
||
pipenv shell | ||
|
||
Create your driver file: | ||
|
||
Create your driver file: |
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.
redundent space attheend of the line
Hi @sleviim @sfishbai ,
This pr is trying to follow the notes for #309 and raised the issue #427 .
One way of solving the issue is creating a requirements_dev.txt with the needed dependencies.
The question is, would you prefer to lock all the module versions or only
idna
(the conflicting one)?