Skip to content

Update multiversion documentation#827

Merged
dgarcia360 merged 6 commits intoscylladb:masterfrom
dgarcia360:update-multiversion-documentation
Jul 4, 2023
Merged

Update multiversion documentation#827
dgarcia360 merged 6 commits intoscylladb:masterfrom
dgarcia360:update-multiversion-documentation

Conversation

@dgarcia360
Copy link
Copy Markdown
Collaborator

  • Improve multiversion command documentation.
  • Identify all the prerequistes necessary to run the software.
  • Add WSL as an alternative terminal to Git Bash.

@dgarcia360 dgarcia360 requested a review from annastuchlik June 23, 2023 17:06
@annastuchlik
Copy link
Copy Markdown
Collaborator

@dgarcia360 The docs look great, but I can't apply the procedure. Let's discuss this.

Comment thread docs/source/configuration/multiversion.rst Outdated
Comment thread docs/source/commands.rst

2. The previous command should generate HTML docs under the ``docs/_build/dirhtml`` directory.

.. tip:: If the command raises an error, see :ref:`Troubleshooting <No_Matching_Refs_Found>` for help.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This PR removes troubleshooting no-maching-refs-found problem, but this is exactly what happened to me during the procedure.
image

@annastuchlik
Copy link
Copy Markdown
Collaborator

@dgarcia360 Are you going to make any updates to this PR? For example, remove info about Git Bash?

…ersion-documentation' of github.com:dgarcia360/sphinx-scylladb-theme-1 into update-multiversion-documentation
@dgarcia360
Copy link
Copy Markdown
Collaborator Author

@annastuchlik PR updated

@dgarcia360 dgarcia360 requested a review from annastuchlik July 4, 2023 08:47
Copy link
Copy Markdown
Collaborator

@annastuchlik annastuchlik left a comment

Choose a reason for hiding this comment

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

LGTM

@dgarcia360 dgarcia360 merged commit 54a3e9a into scylladb:master Jul 4, 2023
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