-
Notifications
You must be signed in to change notification settings - Fork 28
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
Develop docker #21
base: develop
Are you sure you want to change the base?
Develop docker #21
Conversation
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 changed the base branch from master
to develop
.
I believe pull requests should always be merged into develop
.
I'll add a CONTRIBUTING file to explain this point.
Dockerfile/Dockerfile
Outdated
RUN apt install -y gfortran python3 python3-setuptools\ | ||
python3-numpy python3-dev unzip ca-certificates | ||
|
||
COPY sandy-develop.zip / |
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.
Instead of using COPY
and a zip file, install git
and run a clone command. 😉
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.
Sorry for the late response, I will change the Dockerfile and push the update!
Hello! I have finally updated the Dockerfile as requested. |
Hello! I have created a Dockerfile and attached a short text file with build instructions. Best, Jan