Enabled naming style check for FEM, ONNX FE (#6970)

* Enabled naming style check for FEM, ONNX FE

* Used custom version of ncc

* Added self-check

* Removed submdoule

* Moved to public ncc
This commit is contained in:
Ilya Lavrenov
2021-08-09 15:38:08 +03:00
committed by GitHub
parent 461584ffb1
commit 6a63cb9122
13 changed files with 99 additions and 78 deletions

View File

@@ -28,13 +28,15 @@ jobs:
cmake -DENABLE_PYTHON=ON -DENABLE_TESTS=ON -DENABLE_PROFILING_ITT=ON -DSELECTIVE_BUILD=COLLECT ..
- name: Check code style
run: cmake --build build --target clang_format_check_all -j8
run: cmake --build . --target clang_format_check_all -j8
working-directory: build
- name: Create code style diff
if: failure()
run: |
cmake --build build --target clang_format_fix_all
cmake --build . --target clang_format_fix_all
git diff > code_style_diff.diff
working-directory: build
- uses: actions/upload-artifact@v2
if: failure()
@@ -53,15 +55,13 @@ jobs:
run: sudo apt --assume-yes install shellcheck
- name: Install dependencies
run: |
sudo apt --assume-yes install libusb-1.0-0-dev
python3 -m pip install -r ./inference-engine/ie_bridges/python/requirements.txt
run: python3 -m pip install -r ./inference-engine/ie_bridges/python/requirements.txt
- name: CMake
run: |
mkdir build
cd build
cmake ..
cmake -DENABLE_VPU=OFF ..
- name: ShellCheck
run: cmake --build . --target ie_shellcheck -j8
@@ -75,17 +75,16 @@ jobs:
submodules: recursive
- name: Install Clang dependency
run: sudo apt --assume-yes install libusb-1.0-0-dev libclang-9-dev
run: sudo apt --assume-yes install libclang-9-dev
- name: Install Python-based dependencies
run: |
python3 -m pip install pyyaml clang==9.0
run: python3 -m pip install -r cmake/developer_package/ncc_naming_style/requirements_dev.txt
- name: CMake
run: |
mkdir build
cd build
cmake ..
cmake -DENABLE_VPU=OFF ..
- name: Naming convention check
run: cmake --build . --target ncc_all -j8