Thank you for considering contributing to Salah!
- Describe what you expected to happen.
- If possible, include a minimal, complete, and verifiable example to help us identify the issue. This also helps check that the issue is not with your own code.
- Describe what actually happened. Include the full traceback if there was an exception.
- Include tests if your patch is supposed to solve a bug, and explain clearly under which circumstances the bug happens. Make sure the test fails without your patch.
- Make sure all commits are verified.
- Make sure there are no trailing spaces in the any of the modified files.
- Create a branch to identify the issue you would like to work on (e.g.
2287-dry-test-suite
) - Using your favorite editor, make your changes, committing as you go.
- Make sure there are no trailing spaces in the any of the modified files.
- Include tests that cover any code changes you make. Make sure the test fails without your patch.
- Push your commits to GitHub and create a pull request.
- Celebrate 🎉
Run the test suite with:
pytest --cov-report term-missing --cov=flux tests/
In order to build the doc you need to have Sphinx 1.8 or later installed. If you have it or after installing it you can generate the documentation with:
cd docs
make html
This will create the documentation in the docs/build/html
directory.