mirror of
https://github.com/akuker/RASCSI.git
synced 2024-11-25 20:33:35 +00:00
52c2aa474f
* Rebrand project to PiSCSI - rascsi ->piscsi - rasctl -> scsictl - rasdump -> scsidump - ras* -> piscsi* (rasutil -> piscsi_util, etc.) * Refined the formatting and wording of the app startup banner * Kept some references to rascsi and rasctl where backwards compatibility is concerned * Point to the new github repo URL Co-authored-by: nucleogenic <nr@nucleogenic.com> Co-authored-by: Uwe Seimet <Uwe.Seimet@seimet.de>
42 lines
1.1 KiB
Markdown
42 lines
1.1 KiB
Markdown
# PiSCSI Python Apps
|
|
|
|
This directory contains Python-based clients for PiSCSI as well as common
|
|
packages that are shared among the clients.
|
|
|
|
The following paragraphs in this README contain instructions that are shared
|
|
among all Python apps.
|
|
|
|
## Supported Python interpreter
|
|
|
|
The policy in this project is to support the Python 3 interpreter that comes
|
|
standard with the current stable, as well as previous stable release of Debian.
|
|
|
|
At the time of writing they are:
|
|
- Python 3.9.2 in [Debian Bullseye](https://packages.debian.org/bullseye/python3)
|
|
- Python 3.7.3 in [Debian Buster](https://packages.debian.org/buster/python3)
|
|
|
|
## Static analysis with pylint
|
|
|
|
It is recommended to run pylint against new code to protect against bugs
|
|
and keep the code readable and maintainable.
|
|
The local pylint configuration lives in .pylintrc.
|
|
In order for pylint to recognize venv libraries, the pylint-venv package is required.
|
|
|
|
```
|
|
sudo apt install pylint3
|
|
sudo pip install pylint-venv
|
|
source venv/bin/activate
|
|
pylint3 python_source_file.py
|
|
```
|
|
|
|
Examples:
|
|
```
|
|
# check a single file
|
|
pylint web/src/web.py
|
|
|
|
# check the python modules
|
|
pylint common/src
|
|
pylint web/src
|
|
pylint oled/src
|
|
```
|