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

Fix docstring markup in sage/plot and sage/graphs #34164

Closed
kwankyu opened this issue Jul 12, 2022 · 7 comments
Closed

Fix docstring markup in sage/plot and sage/graphs #34164

kwankyu opened this issue Jul 12, 2022 · 7 comments

Comments

@kwankyu
Copy link
Collaborator

kwankyu commented Jul 12, 2022

Part of #34157:

sage/plot/polygon.py:99:1: RST215 Inline interpreted text or phrase reference start-string without end-string.
sage/plot/polygon.py:113:1: RST215 Inline interpreted text or phrase reference start-string without end-string.
sage/graphs/dot2tex_utils.py:89:1: RST201 Block quote ends without a blank line; unexpected unindent.
sage/graphs/dot2tex_utils.py:91:1: RST301 Unexpected indentation.
sage/graphs/graph_plot.py:50:1: RST299 The "csv-table" directive requires content; none supplied.

Component: documentation

Author: Frédéric Chapoton

Branch/Commit: c088449

Reviewer: Matthias Koeppe

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

@kwankyu kwankyu added this to the sage-9.7 milestone Jul 12, 2022
@kwankyu

This comment has been minimized.

@fchapoton
Copy link
Contributor

Branch: u/chapoton/34164

@fchapoton
Copy link
Contributor

Commit: c088449

@fchapoton
Copy link
Contributor

comment:2

done and checked

there remains a failure in graphs, about csv directive lacking content. This is not a true error, as we dynamically fill the content. This need to be excluded from the checks somehow.


New commits:

c088449rst fixes in plot and graphs

@fchapoton
Copy link
Contributor

Author: Frédéric Chapoton

@mkoeppe
Copy link
Member

mkoeppe commented Jul 13, 2022

Reviewer: Matthias Koeppe

@vbraun
Copy link
Member

vbraun commented Jul 28, 2022

Changed branch from u/chapoton/34164 to c088449

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants