1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
|
# Contributing
Thank you for your interest in `tools-python`. The project is open-source software, and bug reports, suggestions, and
most especially patches are welcome.
## Issues
`tools-python` has a [project page on GitHub](https://github.com/spdx/tools-python/) where you
can [create an issue](https://github.com/spdx/tools-python/issues/new) to report a bug, make a suggestion, or propose a
substantial change or improvement. You may also wish to contact the SPDX working group technical team through its
mailing list, [spdx-tech@lists.spdx.org](mailto:spdx-tech@lists.spdx.org).
If you would like to work on a fix for any issue, please assign the issue to yourself or write a comment indicating your
intention prior to creating a patch.
## Development process
We use the GitHub flow that is described here: https://guides.github.com/introduction/flow/
Here's the process to make changes to the codebase:
1. Find or [file an issue](#issues) you'd like to address. Every change should be made to fix or close an issue. Please
try to keep issues reasonably small, focusing on one aspect, or split off sub-issues if possible. Large pull requests
that fix many things at the same time tend to cause a lot of conflicts.
2. Review [open pull requests](https://github.com/spdx/tools-python/pulls) before committing time to a substantial
revision. Work along similar lines may already be in progress.
3. Fork the repository as described [here](https://docs.github.com/en/get-started/quickstart/fork-a-repo#forking-a-repository)
and optionally follow the further steps described to sync your fork and the original repository.
4. Create a new branch in your fork and set up environment:
```sh
git checkout -b fix-or-improve-something
python -m venv ./venv
./venv/bin/activate
pip install -e ".[development]"
```
Note: By using the group `[development]` for the installation, all dependencies (including optional ones) will be
installed. This way we make sure that all tests are executed.
5. Make some changes and commit them to the branch:
```sh
git commit --signoff -m 'description of my changes'
```
#### Licensing
Please sign off in each of your commits that you license your contributions under the terms
of [the Developer Certificate of Origin](https://developercertificate.org/). Git has utilities for signing off on
commits: `git commit -s` or `--signoff` signs a current commit, and `git rebase --signoff <revision-range>`
retroactively signs a range of past commits.
6. Test your changes:
```sh
pytest -vvs # in the repo root
```
7. Check your code style. When opening a pull request, your changes will automatically be checked with `isort`, `black`
and `flake8` to make sure your changes fit with the rest of the code style.
```sh
# run the following commands in the repo root
isort src tests
black src tests
flake8 src tests
```
`black` and `isort` will automatically format the code and sort the imports. The configuration for these linters
can be found in the `pyproject.toml`. `flake8` lists all problems found which then need to be resolved manually.
The configuration for the linter can be found in the `.flake8` file.
8. Push the branch to your fork on GitHub:
```sh
git push origin fix-or-improve-something
```
9. Make a pull request on GitHub.
10. Continue making more changes and commits on the branch, with `git commit --signoff` and `git push`.
11. When done, write a comment on the PR asking for a code review.
12. Some other developer will review your changes and accept your PR. The merge should be done with `rebase`, if
possible, or with `squash`.
13. The temporary branch on GitHub should be deleted (there is a button for deleting it).
14. Delete the local branch as well:
```sh
git checkout master
git pull -p
git branch -a
git branch -d fix-or-improve-something
```
# How to run tests
The tests framework is using pytest:
```
pip install pytest
pytest -vvs
```
|
