Contributing to pyEQL¶
Reporting Issues¶
You can help the project simply by using pyEQL and comparing the output to experimental data and/or other models and tools. If you encounter any bugs, packaging issues, feature requests, comments, or questions, please report them using the issue tracker on github.
Contributing Code¶
To contribute bug fixes, documentation enhancements, or new code, please fork pyEQL and send us a pull request. It’s not as hard as it sounds! Beginning with version 0.6.0, we follow the GitHub flow workflow model.
Hacking pyEQL, step by step¶
Fork the pyEQL repository on Github
Clone your repository to a directory of your choice:
git clone https://github.com/<username>/pyEQL
Install the package and the test dependencies by running the following command from the repository directory:
pip install -e '.[testing]``
Create a branch for your work. Preferably, start your branch name with “feature-”, “fix-”, or “doc-” depending on whether you are contributing bug fixes, documentation or a new feature, e.g. prefix your branch with “fix-” or “doc-” as appropriate:
git checkout -b mybranch
or
git checkout -b doc-mydoc
or
git checkout -b feature-myfeature
Make changes to the code until you’re satisfied.
Push your work back to Github:
git push origin feature-myfeature
Create a pull request with your changes. See this tutorial for instructions.
Guidelines¶
Please abide by the following guidelines when contributing code to pyEQL
:
All changes you make to quacc should be accompanied by unit tests and should not break existing tests. To run the full test suite, run
pytest tests/
from the repository directory.Code coverage should be maintained or increase. Each PR will report code coverage after the tests pass, but you can check locally using pytest-cov, by running
pytest --cov tests/
All code should include type hints and have internally consistent documentation for the inputs and outputs.
Use Google style docstrings
Lint your code with
ruff
by runningruff check --fix src/
from the repo directory. Alternatively, you can install thepre-commit
hooks by runningpre-commit install
from the repository directory. This will prevent committing new changes until all linting errors are fixed.Update the
CHANGELOG.md
file.Ask questions and be open to feedback!
Documentation¶
Improvements to the documentation are most welcome! Our documentation system uses sphinx
with the Materials for Sphinx theme. To edit the documentation locally, run tox -e autodocs
from the repository root directory. This will serve the documents to http://localhost:8000/ so you can view them in your web browser. When you make changes to the files in the docs/
directory, the documentation will automatically rebuild and update in your browser (you might have to refresh the page to see changes).
Changelog¶
We keep a CHANGELOG.md
file in the base directory of the repository. Before submitting your PR, be sure to update the CHANGELOG.md
file under the “Unreleased” section with a brief description of your changes. Our CHANGELOG.md
file lossely follows the Keep a Changelog format, beginning with v0.6.0
.