First off, thank you for considering contributing to xtb-python
.
Please take a moment to review this guidelines to make the contribution process
simple and effective for all involved.
Respecting these guidelines helps communicate that you respect the time of the developers who manage and develop this open source project. In return, they should return this respect by addressing your problem, evaluating changes, and helping you handle your pull requests.
A bug is a demonstratable problem caused by the code in this repository. Good bug reports are extremely valuable for us - thank you!
Before opening a bug report:
- Check if the issue has already been reported.
- Check if it still is an issue or has already been fixed?
Try to reproduce it with the latest version from the
master
branch. - Isolate the problem and create a reduced test case.
A good bug report should not leave others needing to chase you up for more information. So please try to be as detailed as possible in your report, answer at least these questions:
- Which version of
xtb-python
are you using? The current version is always a subject to change, so be more specific. - What is your environment (your laptop, the cluster of the university)?
- What steps will reproduce the issue? We have to reproduce the issue, so we need all the input files.
- What would be the expected outcome?
- What did you see instead?
All these details will help people to fix any potential bugs.
Feature requests are welcome. But take a moment to find out if your idea fits the scope and goals of the project. It is up to you to provide a strong argument to convince the project's developers of the benefits of this feature. Please provide as much detail and context as possible.
Contributions are welcome via GitHub pull requests.
- Each pull request should implement one feature or fix one bug. If you want to add or fix more than one thing, submit more than one pull request.
- Do not commit changes to files that are irrelevant to your feature or
bugfix (e.g.
.gitignore
). - Be willing to accept criticism and work on improving your code.
- We encourage and enforce high quality code (most of the checks are automatically
done for you when submitting a PR):
- Add tests for your new features or fixed bugs such that we can ensure that they stay functional and useful
- Use a linter to catch errors before they become bugs,
either
flake8
orpylint
are perfect tools for this job - Typehints are mandatory and will be automatically checked with
mypy
- All code (except for the tests) is formatted with
black
If you never created a pull request before, welcome 🎉. You can learn how from this great tutorial.
Don't know where to start? You can start by looking through these help-wanted issues.
The sign-off is a simple line at the end of the explanation for a commit. All commits needs to be signed. Your signature certifies that you wrote the patch or otherwise have the right to contribute the material. The rules are pretty simple, if you can certify the below (from developercertificate.org):
Developer Certificate of Origin Version 1.1 Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129 Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed. Developer's Certificate of Origin 1.1 By making a contribution to this project, I certify that: (a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or (b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or (c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it. (d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.
Then you just add a line to every git commit message:
Signed-off-by: Joe Smith <joe.smith@example.com>
Use your real name (sorry, no pseudonyms or anonymous contributions.)
If you set your user.name
and user.email
git configs, you can sign your
commit automatically with git commit -s
.