Skip to content
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

DOC: update the image name for docker image #6143

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

unkcpz
Copy link
Member

@unkcpz unkcpz commented Oct 9, 2023

No description provided.

@unkcpz unkcpz requested a review from sphuber October 9, 2023 13:11
@unkcpz unkcpz force-pushed the doc/xx/installation-docker branch from 005a415 to 5a57c00 Compare October 9, 2023 13:11
@unkcpz
Copy link
Member Author

unkcpz commented Oct 9, 2023

It is a chick-egg problem if this PR merged first (then the latest tag will not exist) or release the v2.5.0 first (the doc needs to update afterward but the v2.5.0 doc will not contain this section, which I think it is not a big deal. @sphuber please find which order you think is more proper.

@@ -19,29 +19,19 @@ This image contains a fully pre-configured AiiDA environment which makes it part

To install Docker, please refer to the `official documentation <https://docs.docker.com/get-docker/>`__.

.. note::

If you are using Linux, you need to have root privileges to do `post-installation steps for the Docker Engine <https://docs.docker.com/engine/install/linux-postinstall/>`__.
Copy link
Member Author

Choose a reason for hiding this comment

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

This is not required anymore since for the linux the official docker doc also recommend to use docker desktop.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it is fine to have the latest docs to point to a release that does not exist yet. We will be releasing soon.

@sphuber sphuber merged commit 5ca609b into aiidateam:main Oct 10, 2023
2 checks passed
@unkcpz unkcpz deleted the doc/xx/installation-docker branch October 10, 2023 08:54
sphuber pushed a commit that referenced this pull request Nov 14, 2023
It was still pointing to the old name instead of the new `aiida-core-with-services`.

Cherry-pick: 5ca609b
sphuber pushed a commit that referenced this pull request Nov 15, 2023
It was still pointing to the old name instead of the new `aiida-core-with-services`.

Cherry-pick: 5ca609b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants