-
Notifications
You must be signed in to change notification settings - Fork 54
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #162 from DeepSpace2/devel
4.2 release
- Loading branch information
Showing
16 changed files
with
300 additions
and
146 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,39 @@ | ||
Thanks for contributing. If creating a pull request please make sure to submit it to the `devel` branch and not to `master`. | ||
Happy coding! | ||
### Thanks for contributing! | ||
|
||
# Branch | ||
Create the pull request against the `devel` branch instead of the `master`. | ||
|
||
# Quality | ||
All pull requests are welcome regardless of quality. We will work together to review and improve the pull request. | ||
|
||
Of course there are some steps that could be considered to improve the quality of the pull request and to support the process of review and development. | ||
|
||
- Make sure your code follows [PEP 8 — Style Guide for Python Code](https://pep8.org/). You can use some [linting tools](https://en.wikipedia.org/wiki/Lint_(software)) to automate the process (e.g. [`pycodestyle`](https://pycodestyle.pycqa.org) or [`flake8`](https://github.com/pycqa/flake8)). | ||
- Test your code using the existing test cases. For details read further. | ||
- It would be great if you could create [`unittest`](https://docs.python.org/3/library/unittest.html)'s for your code. | ||
|
||
# Unittesting | ||
|
||
This project uses Python's default [`unittest`](https://docs.python.org/3/library/unittest.html) package. You can run all test cases via the *discover* feature when you run this in the projects root folder. | ||
|
||
```sh | ||
python3 -m unittest discover | ||
``` | ||
|
||
The output should look like this | ||
```sh | ||
---------------------------------------------------------------------- | ||
Ran 74 tests in 0.823s | ||
|
||
OK | ||
``` | ||
|
||
If you want to run a specific test case or method you need to *install* the package first. It is recommended to use a virtual environment and the `--editable` flag of `pip`. Run this in the projects root folder: | ||
```sh | ||
python3 -m pip install --editable . | ||
``` | ||
Please read further to understand the consequences of `--editable`. | ||
- ["pip documentation - Local project installs - Editable installs"](https://pip.pypa.io/en/stable/topics/local-project-installs/#editable-installs) | ||
- ["When would the -e, --editable option be useful with pip install?"](https://stackoverflow.com/q/35064426/4865723) | ||
|
||
### Happy coding! |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
version: 2 | ||
|
||
build: | ||
os: ubuntu-22.04 | ||
tools: | ||
python: "3.11" | ||
|
||
sphinx: | ||
configuration: docs/conf.py | ||
|
||
python: | ||
install: | ||
- requirements: docs/requirements.txt |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,7 @@ | ||
colour | ||
openpyxl>=2.5,<4 | ||
colour>=0.1.5,<0.2 | ||
jsonschema | ||
openpyxl | ||
pandas | ||
sphinx==5.0.1 | ||
xlrd | ||
pandas<3 | ||
xlrd>=1.0.0,<1.3.0 ; python_version<='3.6' | ||
sphinx==7.2.6 | ||
sphinx-rtd-theme |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.