Skip to content

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

Merged
merged 5 commits into from
Nov 29, 2017
Merged

Conversation

encukou
Copy link
Member

@encukou encukou commented Nov 29, 2017

Some documentation on how to contribute and maintain python-ldap.

Incidentally, this
Fixes: #50

@encukou encukou mentioned this pull request Nov 29, 2017
21 tasks
Copy link
Member

@tiran tiran left a comment

Choose a reason for hiding this comment

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

Two nitpicks

for checking things independent of the Python version:

* ``doc`` checks syntax and spelling of the documentation
* ``coverage`` generates a test coverage report
Copy link
Member

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.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for the catch!

If you don't have some of them installed, you will get an
``InterpreterNotFound`` error, which you can ignore::

ERROR: py37: InterpreterNotFound: python3.7
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

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`.
@encukou encukou merged commit a62695e into python-ldap:master Nov 29, 2017
@encukou encukou deleted the contributing branch November 29, 2017 14:14
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