Skip to content
This repository has been archived by the owner on Sep 24, 2021. It is now read-only.

Lr/update readme #46

Merged
merged 5 commits into from
Jan 15, 2020
Merged

Lr/update readme #46

merged 5 commits into from
Jan 15, 2020

Conversation

lrouhier
Copy link
Contributor

Updating the readme for the latest docker release.
Change version to 'latest' after discussion on issue #45.
Can't change the last command because of 'latest' is not an SCT version.

README.rst Outdated
@@ -76,7 +76,7 @@ Installation

.. code:: sh

docker pull neuropoly/sct:sct-4.0.0-official
docker pull neuropoly/sct:sct-latest
Copy link
Member

@jcohenadad jcohenadad Jan 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would recommend to use latest instead of sct-latest for simplicity and according to conventions (see my post)
same comment for all other versions.

@jcohenadad jcohenadad merged commit 8326b6b into master Jan 15, 2020
@jcohenadad jcohenadad deleted the lr/update-readme branch January 15, 2020 23:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants