Skip to content

Conversation

uggla
Copy link
Collaborator

@uggla uggla commented Feb 11, 2016

  • Put in place sphinx mechanism to build documentation.
    Most of the documentation is generated from python docstrings.
  • redfish-client.py usage is a bit modified to display a decent
    documentation.
  • A link rfclient.py to redfish-client.py was created to allow sphinx
    parsing. The '-' char is not allowed for module and so sphinx autodoc
    module.
  • Note : Copyright may need to be reviewed as well as main.py.

- Put in place sphinx mechanism to build documentation.
  Most of the documentation is generated from python docstrings.
- redfish-client.py usage is a bit modified to display a decent
  documentation.
- A link rfclient.py to redfish-client.py was created to allow sphinx
  parsing. The '-' char is not allowed for module and so sphinx autodoc
  module.
- Note : Copyright may need to be reviewed as well as main.py.
# built documents.
#
# The short X.Y version.
version = '0.4'
Copy link
Owner

Choose a reason for hiding this comment

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

Would be good to use PBVER here and to add that file to the files processed by pb in python-redfish.pb.

@bcornec
Copy link
Owner

bcornec commented Feb 12, 2016

The dep on shpinx should also be reported in the rpm/deb dependencies. I'll add a patch for that later on.

bcornec added a commit that referenced this pull request Feb 12, 2016
@bcornec bcornec merged commit a9d349e into bcornec:devel Feb 12, 2016
@uggla
Copy link
Collaborator Author

uggla commented Feb 12, 2016

Hello Bruno,

Thanks for merging so early in the morning ! :)
Haha, I thought about the PBVER stuff right after clicking the validate button of the pull request.
I will fix that in a next commit that should come up very soon.

@uggla uggla deleted the sphinxdoc branch February 13, 2016 20:13
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.

2 participants