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

docs: update theme 1.1 #56

Merged
merged 4 commits into from
Feb 2, 2022
Merged

docs: update theme 1.1 #56

merged 4 commits into from
Feb 2, 2022

Conversation

dgarcia360
Copy link
Contributor

@dgarcia360 dgarcia360 commented Feb 2, 2022

Related issue scylladb/sphinx-scylladb-theme#133

ScyllaDB Sphinx Theme 1.1 is now released 🥳

We’ve made a number of updates to update all our dependencies to the latest version and introduced new directives you can use to write great docs.

You can read more about all notable changes here.

⚠️Breaking changes

Before, the theme installed poetry 1.1.x as a dependency to manage Python dependencies. However, poetry 1.2.x changed the installation method. Therefore, we've decided to #307 Make poetry a prerequisite so that you can decide to install the poetry version you prefer.

To preview the docs locally, you should uninstall the previous version of poetry. Then, install the latest version:

  1. Uninstall Poetry 1.1.x.
curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | POETRY_UNINSTALL=1 python -
  1. Install Poetry 1.2.x. For detailed instructions, see Poetry installation.

How to test this PR

  1. Clone this PR. For more information, see Cloning pull requests locally.

  2. Uninstall poetry 1.1 and install poetry 1.2. For more information, see Breaking changes notice above.

  3. Enter the docs folder, and run:

make preview
  1. Open http://127.0.0.1:5500/ with your favorite browser. The doc should render without errors, and the version should be Sphinx Theme version (see the footer) must be 1.1.x:

image

@dgarcia360
Copy link
Contributor Author

@tzach Please review

Copy link
Collaborator

@tzach tzach left a comment

Choose a reason for hiding this comment

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

few questions

@@ -0,0 +1,31 @@
name: "Docs / Publish"
Copy link
Collaborator

Choose a reason for hiding this comment

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

what are the updates in the action?
Its hard to understand the update when we have a new file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

.github/workflows/docs-pr@v2.yaml Show resolved Hide resolved
docs/conf.py Show resolved Hide resolved
@@ -76,7 +56,6 @@ def setup(sphinx):
'github_issues_repository': 'scylladb/care-pet',
'github_repository': 'scylladb/care-pet',
'hide_edit_this_page_button': 'false',
'hide_sidebar_index': 'true',
Copy link
Collaborator

Choose a reason for hiding this comment

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

why removing this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -76,7 +78,7 @@

.. raw:: html
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we still need HTML in the page?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, we managed to remove one html tag but the issue is still open: scylladb/sphinx-scylladb-theme#217

@tzach
Copy link
Collaborator

tzach commented Feb 2, 2022

failed to build the docs

✔ care-pet/docs [docs-theme-1-1|✔] 
09:03 $ make preview 
/home/tzach/.local/bin/poetry install
make: /home/tzach/.local/bin/poetry: No such file or directory
make: *** [Makefile:30: setup] Error 127

@dgarcia360
Copy link
Contributor Author

failed to build the docs

Please see Breaking changes.

@tzach tzach merged commit 8318dde into scylladb:master Feb 2, 2022
@dgarcia360 dgarcia360 deleted the docs-theme-1-1 branch February 9, 2022 10:55
@dgarcia360 dgarcia360 changed the title Update docs theme 1.1 docs: update theme 1.1 Feb 11, 2022
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.

None yet

2 participants