search

espressif / esp-usb

Other
15 stars 9 forks source link

CI: Add Clang tidy #25

Closed peter-marcisovsky closed 2 months ago

peter-marcisovsky commented 3 months ago

Adding Clang tidy related files.

We wanted to reuse Clang tidy CI job from idf-extra-components, where Clang tidy job was run from its main test_app. Since we have split the main test_app into into several test apps in this respository an extensive Clang tidy job rework would be needed to run Clang tidy in all the test_apps.

We decided to create a folder clang_tidy, from where Clang tidy is run. Doing so, nearly no modification is needed to be done in the Clang tidy CI job from idf-extra-components

.github/filter_sarif.py - copy from idf-extra-components (no modification) .github/workflows/clang-tidy.yml - copy from idf-extra-components (line 41 added, line 38 modified folder name)

clang_tidy folder name and content of clang_tidy/README.md is opened for discussion.

github-advanced-security[bot] commented 3 months ago

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

tore-espressif commented 2 months ago

@peter-marcisovsky Hit the merge button when ready!