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

DM-41296: Migrate out of documenteer.sphinxext namespace #193

Merged
merged 8 commits into from
Oct 26, 2023

Conversation

jonathansick
Copy link
Member

@jonathansick jonathansick commented Oct 24, 2023

Backwards-incompatible changes

  • Delete the refresh-lsst-bib CLI command. Technotes now automatically vendor lsst-texmf's bib files and cache them using documenteer.ext.githubbibcache.

  • Delete the build-stack-docs CLI command, which was replaced by the stack-docs and package-docs CLIs in Documenteer 0.3.0.

  • The documenteer.conf.pipelines and documenteer.conf.pipelinespkg configuration modules now derive from documenteer.conf.guide. In doing so, the Pipelines documentation configuration works the same as Rubin Guides, but with additional configuration for pipelines-specific Sphinx extensions and other configurations. With this change, the lsst-sphinx-bootstrap-theme is no longer used by Documenteer.

  • The documenteer.sphinxext module has been removed and the existing Sphinx extensions within it are now available from documenteer.ext. It's no longer possible to use documenteer.sphinxext to automatically load all Documenteer Sphinx extensions. Extensions need to now be added individually to the extensions configuration variable in conf.py. The migrated extension modules are:

    • documenteer.ext.bibtex
    • documenteer.ext.jira
    • documenteer.ext.lsstdocushare
    • documenteer.ext.lssttasks
    • documenteer.ext.mockcoderefs
    • documenteer.ext.packagetoctree

New features

  • Add sphinx-jinja to the Rubin guides configuration by default.
  • Add sphinx-rediraffe to the Rubin guides configuration by default.

@jonathansick jonathansick force-pushed the tickets/DM-41296 branch 3 times, most recently from 09b5f78 to 0e3c99a Compare October 26, 2023 16:41
- Migrate the bibtex, jira, lsstdocushare, mockcoderefs, packagetoctree
  and lssttasks Sphinx extensions for documenteer.sphinxext to
  documenteer.ext, where all new extensions are being added.

- Drop the original documenteer.sphinxconfig package; all Pipelines
  configurations are now done through the documenteer.conf.pipelines
  configuration module.
This ensures that all guides automatically include the sphinx-jinja
dependency. They will need to configure their context in their own
conf.py files, though.
Now documenteer.conf.pipelines imports its base configuration from
documenteer.conf.guide. This allows us to retire the original theme,
lsst-sphinx-bootstrap-theme. It'll also help us unify the pipelines
documentation with the overall engineering and support we have for Rubin
guides in general.

The pipelines configuration add additional extensions. It still makes
sense to ship this as a configuration through documenteer because
individual packages will use documenteer.conf.pipelinespkg to do single
package documentation previews.

Since the guides configuration uses documenteer.toml, it might make
sense to migrate manifest.yaml into documenteer.toml as a separate
table.
This CLI was developed for the original technotes to vendor bibtex files
from lsst/lsst-texmf. Documenteer's technotes now download and cache bib
files automatically.
This CLI was the predecessor for stack-docs. It should be fully
deprecated at this point.
This makes it convenient to create redirect pages when documentation is
moved or renamed.

https://sphinxext-rediraffe.readthedocs.io/en/latest/
This is the initial move with rediraffe redirects. More work is needed
to make the content reflect the new relationship between guides and the
specific pipelines docs use case.
@jonathansick jonathansick merged commit 4a53207 into main Oct 26, 2023
5 checks passed
@jonathansick jonathansick deleted the tickets/DM-41296 branch October 26, 2023 18:14
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.

1 participant