Skip to content

Add better docstrings and move doc generation to Sphinx#105

Merged
jammons merged 4 commits intoslackapi:masterfrom
jammons:master
Jul 7, 2016
Merged

Add better docstrings and move doc generation to Sphinx#105
jammons merged 4 commits intoslackapi:masterfrom
jammons:master

Conversation

@jammons
Copy link
Copy Markdown
Contributor

@jammons jammons commented Jul 1, 2016

Added docstrings to most public methods and we should now be able to generate docs by using sphinx.

Also moved doc/ to docs/

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 1, 2016

Coverage Status

Coverage remained the same at 59.916% when pulling 6f907d4 on jammons:master into 4fc299f on slackhq:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 1, 2016

Coverage Status

Coverage remained the same at 59.916% when pulling 6d0fef3 on jammons:master into 4fc299f on slackhq:master.

Comment thread LICENSE Outdated
@@ -1,3 +1,7 @@
The MIT License (MIT)

Copyright (c) 2015 Slack Technologies, Inc
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should probably be 2015–2016

Jeff Ammons added 3 commits July 7, 2016 15:14
Goal of this commit was to lay some groundwork for continuing to add
more robust docs. This should set us up to have more thorough
docstring-based docs inside the code as well as to have better organized
documentation around how to use this library and how to contribute.
docs/ seems to be the more standard naming convention for python
projects.

Also removed unused .rst file
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 7, 2016

Coverage Status

Coverage remained the same at 61.265% when pulling 1674bab on jammons:master into 18cb7ca on slackhq:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.4%) to 60.87% when pulling 22fc7a1 on jammons:master into 18cb7ca on slackhq:master.

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 7, 2016

Coverage Status

Coverage remained the same at 61.265% when pulling 22fc7a1 on jammons:master into 18cb7ca on slackhq:master.

@jammons jammons merged commit d367003 into slackapi:master Jul 7, 2016
@jammons jammons mentioned this pull request Jul 9, 2016
c-goosen pushed a commit to c-goosen/python-slackclient that referenced this pull request Jun 18, 2019
Add better docstrings and move doc generation to Sphinx
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