cpp-linter / cpp-linter-action

A Github Action for linting C/C++ code integrating clang-tidy and clang-format to collect feedback provided in the form of file-annotations, thread-comments, workflow step-summary, and Pull Request reviews.
https://cpp-linter.github.io/cpp-linter-action/
MIT License
87 stars 20 forks source link

Assess push permission #226

Closed 2bndy5 closed 3 months ago

2bndy5 commented 3 months ago

This verifies and updates the permissions needed for certain features about push events.

Some update about token permissions must have recently rolled out across all of github because the REST API docs have changed and some noted permission scopes have been changed...

github-actions[bot] commented 3 months ago

Cpp-Linter Report :warning:

Some files did not pass the configured checks!

clang-format reports: 2 file(s) not formatted - docs/examples/demo/demo.cpp - docs/examples/demo/demo.hpp
clang-tidy reports: 7 concern(s) - **docs/examples/demo/demo.cpp:3:10:** warning: [[modernize-deprecated-headers](https://clang.llvm.org/extra/clang-tidy/checks/modernize/deprecated-headers.html)] > inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead ```cpp #include ^~~~~~~~~ ``` - **docs/examples/demo/demo.cpp:8:5:** warning: [[modernize-use-trailing-return-type](https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-trailing-return-type.html)] > use a trailing return type for this function ```cpp int main(){ ~~~ ^ auto -> int ``` - **docs/examples/demo/demo.cpp:10:13:** warning: [[readability-braces-around-statements](https://clang.llvm.org/extra/clang-tidy/checks/readability/braces-around-statements.html)] > statement should be inside braces ```cpp for (;;) break; ^ { ``` - **docs/examples/demo/demo.cpp:13:5:** warning: [[cppcoreguidelines-pro-type-vararg](https://clang.llvm.org/extra/clang-tidy/checks/cppcoreguidelines/pro-type-vararg.html)] > do not call c-style vararg functions ```cpp printf("Hello world!\n"); ^ ``` - **docs/examples/demo/demo.hpp:6:11:** warning: [[modernize-use-default-member-init](https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html)] > use default member initializer for 'useless' ```hpp char* useless; ^ {"\0"} ``` - **docs/examples/demo/demo.hpp:7:9:** warning: [[modernize-use-default-member-init](https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-default-member-init.html)] > use default member initializer for 'numb' ```hpp int numb; ^ {0} ``` - **docs/examples/demo/demo.hpp:11:11:** warning: [[modernize-use-trailing-return-type](https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-trailing-return-type.html)] > use a trailing return type for this function ```hpp void *not_useful(char *str){useless = str;} ~~~~~~^ auto -> void * ```

Have any feedback or feature suggestions? Share it here.