91634c3c9b
flake8 was used to evaluate the file, with a couple of exeptions:
E501,E266,W503
black was used to reformat the code automatically
The changes were:
* Added a README that explains how to use pipenv and pip,
* Remove unnecessary return statements,
* Remove '\' from the end of the lines,
* Use f-strings (>= 3.6) since we are requiring Python 3.7,
* Commenting unused variables,
* Adding assert when Python >= 3.7 is not being used,
* Wrapping long lines to 100 (Qt Style),
* Re-factoring some lines,
* Re-ordering imports,
* Naming `except` for sympy (SympifyError, TypeError)
Change-Id: Ie05f754e7d8ee4bf427117c58e0eb1b903202933
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
55 lines
1.6 KiB
Markdown
55 lines
1.6 KiB
Markdown
# CMake Utils
|
|
|
|
This directory holds scripts to help the porting process from `qmake` to `cmake` for Qt6.
|
|
|
|
# Requirements
|
|
|
|
* [Python 3.7](https://www.python.org/downloads/),
|
|
* `pipenv` or `pip` to manage the modules.
|
|
|
|
## Python modules
|
|
|
|
Since Python has many ways of handling projects, you have a couple of options to
|
|
install the dependencies of the scripts:
|
|
|
|
### Using `pipenv`
|
|
|
|
The dependencies are specified on the `Pipfile`, so you just need to run
|
|
`pipenv install` and that will automatically create a virtual environment
|
|
that you can activate with a `pipenv shell`.
|
|
|
|
### Using `pip`
|
|
|
|
It's highly recommended to use a [virtualenvironment](https://virtualenv.pypa.io/en/latest/)
|
|
to avoid conflict with other packages that are already installed: `pip install virtualenv`.
|
|
|
|
* Create an environment: `virtualenv env`,
|
|
* Activate the environment: `source env/bin/activate`
|
|
(on Windows: `source env\Scripts\activate.bat`)
|
|
* Install the requirements: `pip install -r requirements.txt`
|
|
|
|
# Contributing to the scripts
|
|
|
|
You can verify if the styling of a script complaint with PEP8, with a couple of exceptions:
|
|
|
|
Install [flake8](http://flake8.pycqa.org/en/latest/) (`pip install flake8`) and run it
|
|
on the script you want to test:
|
|
|
|
```
|
|
flake8 <file>.py --ignore=E501,E266,W503
|
|
```
|
|
|
|
* `E501`: Line too long (82>79 characters),
|
|
* `E266`: Too many leading '#' for block comment,
|
|
* `W503`: Line break occurred before a binary operator)
|
|
|
|
You can also modify the file with an automatic formatter,
|
|
like [black](https://black.readthedocs.io/en/stable/) (`pip install black`),
|
|
and execute it:
|
|
|
|
```
|
|
black -l 100 <file>.py
|
|
```
|
|
|
|
Using Qt's maximum line length, 100.
|