-
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
Tagged Installation Instructions #840
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.
Looks good, will you also update the developer docs for making a new release
|
||
``` | ||
git clone https://github.com/angelolab/ark-analysis.git | ||
git clone https://github.com/angelolab/ark-analysis.git --depth 1 --branch v0.4.4 |
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.
Can you try it out to make sure this works as expected, i.e. previous docker gets pulled and specified from start_docker.sh script, etc etc
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.
Yeah the VERSION
in start_docker_sh
is v0.4.4
, so it runs as expected.
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.
Looks good!
If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #830.
How did you implement your changes
Added directions to install a specific release of Ark.
Remaining issues
N/A.