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

reStructuredText format output #440

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

paweld2
Copy link

@paweld2 paweld2 commented Mar 29, 2021

A proposition for reStructuredText format output.

The output is compatible with sphinx. No content table is provided, because it is generated by sphinx.

HTML and Latex output tested.

The references and links are correctly matched:
".. _ref_NAME:" matches ":ref:NAME"

The tables are created in the CVS format.

It is not possible to create references to table row values, that is why for each scalar value a separate subsection is created.

@paweld2
Copy link
Author

paweld2 commented Mar 29, 2021

@pseudomuto please review.

I know that custom templates are supported, but I assume that solving the specific reStructuredText issues has value for more that one person.

@kumare3
Copy link

kumare3 commented May 21, 2021

@paweld2 we are interested in rst support for the flyte.org project. Let us know if we can help test
cc @pmahindrakar-oss

@kumare3
Copy link

kumare3 commented May 23, 2021

@paweld2 we used this for generating Flyte.org docs and works beautifully. @pseudomuto what would the process to get this checked in?

@@ -67,6 +68,7 @@ examples: build tmp/googleapis examples/proto/*.proto examples/templates/*.tmpl
@$(EXAMPLE_CMD) --doc_opt=html,example.html:Ignore* examples/proto/*.proto
@$(EXAMPLE_CMD) --doc_opt=json,example.json:Ignore* examples/proto/*.proto
@$(EXAMPLE_CMD) --doc_opt=markdown,example.md:Ignore* examples/proto/*.proto
@$(EXAMPLE_CMD) --doc_opt=restructuredtext,example.srt:Ignore* examples/proto/*.proto
Copy link

Choose a reason for hiding this comment

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

Suggested change
@$(EXAMPLE_CMD) --doc_opt=restructuredtext,example.srt:Ignore* examples/proto/*.proto
@$(EXAMPLE_CMD) --doc_opt=restructuredtext,example.rst:Ignore* examples/proto/*.proto

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.

3 participants