diff options
author | Diego Russo <diego.russo@arm.com> | 2020-04-21 17:39:10 +0100 |
---|---|---|
committer | Tim Hall <tim.hall@arm.com> | 2020-06-18 17:53:52 +0100 |
commit | e8a10454eba4c7392cb301fbfbe796e5bfb2b729 (patch) | |
tree | 8e54bffd46c55848a86040d3588b5365ef019d8f /README.md | |
parent | 3ec04ac9e38d26193e0081a8e0fa3b8b667bb688 (diff) | |
download | ethos-u-vela-e8a10454eba4c7392cb301fbfbe796e5bfb2b729.tar.gz |
Add reorder-python-import pre-commit hook
Also updated README.md
Change-Id: I118309c61f4d00e8508d6b888c606995490fba39
Signed-off-by: Diego Russo <diego.russo@arm.com>
Diffstat (limited to 'README.md')
-rw-r--r-- | README.md | 11 |
1 files changed, 10 insertions, 1 deletions
@@ -110,6 +110,7 @@ Contributions are accepted under [Apache License 2.0](LICENSE.txt). Only submit ## License Vela is licensed under [Apache License 2.0](LICENSE.txt) + ## Contributions and Pull Requests Contributions are accepted under Apache-2.0. Only submit contributions where you have authored all of the code. @@ -117,7 +118,7 @@ Contributions are accepted under Apache-2.0. Only submit contributions where you ### Sanity checks The Python codebase is PEP8 compliant with the exception of 120 characters line length. -We run black and flake8 against the code base excluding "ethosu/vela/tflite/" and "ethosu/vela/ethos\_u55\_regs" directories because they are auto-generated by third party tools. +We run reorder-python-import, black and flake8 against the code base excluding "ethosu/vela/tflite/" and "ethosu/vela/ethos\_u55\_regs" directories because they are auto-generated by third party tools. Those tools are run using [pre-commit framework](https://pre-commit.com/). The configuration file is .pre-commit-config.yaml #### Install tools @@ -146,9 +147,17 @@ The checks will be run before the commit: if one of them fails, you need to fix Those checks can be run manually. This can be achievied running the following ``` +$ pre-commit run reorder-python-imports --all-files +... $ pre-commit run flake8 --all-files ... $ pre-commit run black --all-files ``` If you don't specify anything after run, it will execute all the checks. +``` +$ pre-commit run --all-files +Reorder python imports...................................................Passed +black....................................................................Passed +flake8...................................................................Passed +``` |