Contributions are welcome, and they are greatly appreciated! Every little bit helps, and credit will always be given.
You can contribute in many ways:
Report bugs at pytest-cookies/issues.
If you are reporting a bug, please include:
- Your operating system name and version.
- Any details about your local setup that might be helpful in troubleshooting.
- Detailed steps to reproduce the bug.
Look through the GitHub issues for bugs. Anything tagged with bug and help wanted is open to whoever wants to implement a fix for it.
Look through the GitHub issues for features. Anything tagged with enhancement and help wanted is open to whoever wants to implement it.
pytest-cookies could always use more documentation, whether as part of the official docs, in docstrings, or even on the web in blog posts, articles, and such.
The best way to send feedback is to file an issue.
If you are proposing a new feature:
- Explain in detail how it would work.
- Keep the scope as narrow as possible, to make it easier to implement.
- Remember that this is a volunteer-driven project, and that contributions are welcome :)
Ready to contribute? Here's how to set up pytest-cookies for local development. Please note this documentation assumes you already have virtualenv and git installed and ready to go.
-
Fork the pytest-cookies repo on GitHub.
-
Clone your fork locally:
$ git clone https://github.com/YOUR-USERNAME/pytest-cookies
-
Assuming you have virtualenv installed (If you have Python 3.6 this should already be there), you can create a new environment for your local development by typing:
$ python -m venv pytest-cookies-env $ source pytest-cookies-env/bin/activate
This should change the shell to look something like (pytest-cookies-env) $
-
Create a branch for local development:
$ git checkout -b name-of-your-bugfix-or-feature
-
Now you can make your changes locally.When you're done making changes, use tox to run checks against your changes. Install it as follows:
$ pip install tox
-
First you want to make sure that your branch passes flake8.
$ tox -e flake8
-
The next step would be to run the tests against your environment (the special tox environment
py
refers to the currently active python environment):$ tox -e py
-
Before raising any pull request you should run tox. This will run the tests across different versions of Python:
$ tox
-
If your contribution is a bug fix or new feature, you may want to add a test to the existing test suite. See section Add a New Test below for details.
-
Commit your changes and push your branch to GitHub:
$ git add . $ git commit -m "Your detailed description of your changes." $ git push origin name-of-your-bugfix-or-feature
-
Submit a pull request through the GitHub website.
Before you submit a pull request, check that it meets these guidelines:
-
The pull request should include tests.
-
If the pull request adds functionality, the docs should be updated. Put your new functionality into a function with a docstring, and add the feature to the list in README.md.
-
The pull request should work for Python 3.6, 3.7, 3.8 and 3.9.
When fixing a bug or adding features, it's good practice to add a test to demonstrate your fix or new feature behaves as expected. These tests should focus on one tiny bit of functionality and prove changes are correct.
To write and run your new test, follow these steps:
-
Add the new test to
tests/test_<file>.py
. Focus your test on the specific bug or a small part of the new feature. -
If you have already made changes to the code, stash your changes and confirm all your changes were stashed:
$ git stash $ git stash list
-
Run your test and confirm that your test fails. If your test does not fail, rewrite the test until it fails on the original code:
$ tox -e py
-
(Optional) Run the tests with tox to ensure that the code changes work with different Python versions:
$ tox
-
Proceed work on your bug fix or new feature or restore your changes. To restore your stashed changes and confirm their restoration:
$ git stash pop $ git stash list
-
Rerun your test and confirm that your test passes. If it passes, congratulations!