sagemath/sage

Improve documentation of sage.features

Closed this issue · 13 comments

We also change the sporadic use of # optional: FEATURE by the more common # optional - FEATURE.

Depends on #32866
Depends on #32174

CC: @seblabbe @saraedum @kwankyu @slel

Component: documentation

Author: Matthias Koeppe

Branch/Commit: 547206c

Reviewer: Sébastien Labbé

Issue created by migration from https://trac.sagemath.org/ticket/32925

Changed dependencies from #32866 to #32866, #32174

Commit: d364333

Last 10 new commits:

12a2507src/sage/features/interfaces.py: Fix indentation in docstring
32dc210Merge #32174
b8eb3ffsrc/doc/en/reference/misc/index.rst: Move sage/features from 'Interactive' to 'Programming Utilities'
322e7c9sage.features: Add headers for the reference manual
0b26010src/doc/en/reference/misc/index.rst: Add sage.features.interfaces
dc954a7src/sage/features/__init__.py: Fix doc markup
fe05309sage.features.pkg_systems: New, move PackageSystem and subclasses here
b7cd28esage.features.sagemath: Improve docstring markup
bd59980sage.features.mip_backends: Improve doc markup
d364333sage.features: Improve doc markup
slel commented
comment:5

Some optional doctests are marked # optional: blah,
rather than # optional - blah as in the developer guide,
not sure whether that matters at all.

comment:6

git grep 'sage:.*# *optional.*:' shows that both styles are used in the Sage library, but I think it's a good idea to make it uniform

Changed commit from d364333 to 547206c

Branch pushed to git repo; I updated commit sha1. New commits:

547206cChange "# optional: FEATURE" to "# optional - FEATURE"

Description changed:

--- 
+++ 
@@ -1 +1,2 @@
+We also change the sporadic use of `# optional: FEATURE` by the more common `# optional - FEATURE`.
 

Reviewer: Sébastien Labbé

comment:9

Works for me. Patchbot confirms that the documentation build fine.

comment:10

Thank you!