-
Notifications
You must be signed in to change notification settings - Fork 127
Doc fixes & contributing guide #57
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
Conversation
0049a79
to
3b186ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two nitpicks
Doc/contributing.rst
Outdated
for checking things independent of the Python version: | ||
|
||
* ``doc`` checks syntax and spelling of the documentation | ||
* ``coverage`` generates a test coverage report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
coverage-report
generates a test coverage report for Python code. It must be used as last test env, e.g.tox -e py27,py36,coverage-report
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the catch!
Doc/contributing.rst
Outdated
If you don't have some of them installed, you will get an | ||
``InterpreterNotFound`` error, which you can ignore:: | ||
|
||
ERROR: py37: InterpreterNotFound: python3.7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tox has skip_missing_interpreters
, https://tox.readthedocs.io/en/latest/config.html#confval-skip_missing_interpreters=BOOL Wanna add it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! Thanks!
All you wanted to know about contributing to python-ldap! The refleak instructions from INSTALL are merged into contribution instructions. The file itself is made to link to installation instructions. Also add a CONTRIBUTING file that references the new text.
This directory is created using `make html`.
3b186ee
to
909c26b
Compare
Some documentation on how to contribute and maintain python-ldap.
Incidentally, this
Fixes: #50