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

chore: use Canonical's sphinx-docs-starter-pack #197

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

tigarmo
Copy link
Collaborator

@tigarmo tigarmo commented Oct 31, 2023

This commit adds Canonical's starter pack for Sphinx documentation. A consequence of this is that the doc-related dependencies are listed in a requirements file, so we drop those optional-deps from pyproject.toml.

Fixes #187

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox?

@tigarmo tigarmo force-pushed the CRAFT-2175-craft-cli-starter-pack branch from 8aa7c6c to d9b188f Compare October 31, 2023 14:03
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #197 (54a895b) into main (a88b353) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #197   +/-   ##
=======================================
  Coverage   94.73%   94.73%           
=======================================
  Files           7        7           
  Lines        1063     1063           
  Branches      194      194           
=======================================
  Hits         1007     1007           
  Misses         53       53           
  Partials        3        3           

@tigarmo tigarmo force-pushed the CRAFT-2175-craft-cli-starter-pack branch 2 times, most recently from f366818 to 0da9e4e Compare October 31, 2023 14:15
This commit adds Canonical's starter pack for Sphinx documentation.

Notes:

- Doc-related dependencies are listed in a requirements file, so we
  drop those optional-deps from pyproject.toml. Extra deps are added
  to docs/.sphinx/requirements.txt.
- The docs/.readthedocs.yaml file is altered to  allow warnings (the
  docs generation from the generates harmless warnings).

Fixes #187
@tigarmo tigarmo force-pushed the CRAFT-2175-craft-cli-starter-pack branch 2 times, most recently from 2b1d15e to 7971bca Compare October 31, 2023 17:03
@tigarmo tigarmo marked this pull request as ready for review October 31, 2023 17:41
@tigarmo tigarmo requested review from mr-cal and syu-w October 31, 2023 17:41
Copy link
Contributor

@syu-w syu-w left a comment

Choose a reason for hiding this comment

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

This one isn't use the git submodule?

@tigarmo
Copy link
Collaborator Author

tigarmo commented Oct 31, 2023

This one isn't use the git submodule?

yeah I figured I'd keep it simple and follow the instructions on the starter-pack site. I hope that next time we "need" to update this it'll be with a proper package hehe

- fix spelling and update custom wordlist
- chore: make linkcheck ignore an internal reference
- fix various yaml formatting issues
- don't run the doc linting workflow twice
- chore: disabled autonamedtuple extension:

The problem: when running in Python3.8, the autonamedtuple extension
(part of sphinx-toolbox) fails to eval newer-style type annotations
(like "type[X]" instead of "typing.Type[X]"), even if the Python module
that has said code has the correct __future__ annotations.

This affects us because the documentation linters on CI run on Ubuntu
20.04, which has Python 3.8 by default. So the quicker fix here is to
disable the extension by instead enabling everything else in the toolbox
manually.
@tigarmo tigarmo force-pushed the CRAFT-2175-craft-cli-starter-pack branch from 7971bca to 54a895b Compare October 31, 2023 20:43
@tigarmo tigarmo merged commit 39473bb into main Oct 31, 2023
10 checks passed
@tigarmo tigarmo deleted the CRAFT-2175-craft-cli-starter-pack branch October 31, 2023 20:57
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.

Update documentation to the official Canonical template
3 participants