Skip to content

Move documentation for how to use this module to the docstring so it's rendered on RTD#672

Open
mthaddon wants to merge 1 commit intojuju:masterfrom
mthaddon:nrpe-docstring
Open

Move documentation for how to use this module to the docstring so it's rendered on RTD#672
mthaddon wants to merge 1 commit intojuju:masterfrom
mthaddon:nrpe-docstring

Conversation

@mthaddon
Copy link
Copy Markdown

Currently the documentation on https://charm-helpers.readthedocs.io/en/latest/api/charmhelpers.contrib.charmsupport.html only renders as one line that's not very useful. Move the documentation from comments lower down in the file to the docstring.

@erik78se
Copy link
Copy Markdown

Might also consider mentioning that this will not build for charms using setuptools > 58 which effectively will break all builds with charmcraft as of today.

So, basically, its broken with charmcraft.

canonical/charmcraft#632 (comment)

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