-
Notifications
You must be signed in to change notification settings - Fork 53
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
Upgrade docstring to Sphinx style. #45
Comments
example of doxygen style:
|
example of Sphinx style:
|
You can check the difference on the automatic documentation: doxygen style is not recognized by Sphinx. |
The general guideline of docstring, http://www.python.org/dev/peps/pep-0257/ It does not specify what exact formatting is preferred. |
This page contains abundant examples of what a Sphinx-style docstring is like. We use the autodoc extension of Sphinx to extract the docstring and format on the website. http://packages.python.org/an_example_pypi_project/sphinx.html |
I am ready to change docstring style, but the question is : ONLY the docstring that in 'doxygen style' should be changed OR the docstring that in some other kind style should be changed?I find some comments are not likely to be docstring. |
@uestcmy , You can start with those docstrings documented in doxygen format and let me merge them. I think giving a trial for the Github workflow is more important. Once done for the first time, you can add other docstrings. You can view some sample pull requests here . As a longer term plan, I think:
|
I'm looking into the problem. Note one ref of Rst, |
I try to use pull request, maybe some mistakes exist. |
Another question is that I cannot render html of docstring in the *.py in app directory ,like "forward" and "mysofa". |
If you want to use automodule directive to autodoc those files, they must form a module first. In Python, this is indicated by a |
One pointer is left in https://github.com/hupili/snsapi/wiki/Coding-convention |
Some original docstring is in Java-style (junbo initialized in doxygen style). We want to change to Sphinx style. First is to conform to Python convention. Second is to help auto documentation.
The text was updated successfully, but these errors were encountered: