fix: remove Debian LLVM PPA repo (Linux runners only)
resolves #265
Users that expect the PPA to be present after running cpp-linter-action can set the environment variable KEEP_DEBIAN_LLVM_PPA to true, on, or 1. By default, the PPA is removed.
Cpp-Linter Report :warning:
Some files did not pass the configured checks!
clang-format (v12.0.0) reports: 2 file(s) not formatted
- docs/examples/demo/demo.cpp
- docs/examples/demo/demo.hpp
clang-tidy (v12.0.0) reports: 7 concern(s)
-
docs/examples/demo/demo.cpp:3:10: warning: [modernize-deprecated-headers]
inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
#include <stdio.h> ^~~~~~~~~ <cstdio> -
docs/examples/demo/demo.cpp:8:5: warning: [modernize-use-trailing-return-type]
use a trailing return type for this function
int main(){ ~~~ ^ auto -> int -
docs/examples/demo/demo.cpp:10:13: warning: [readability-braces-around-statements]
statement should be inside braces
for (;;) break; ^ { -
docs/examples/demo/demo.cpp:13:5: warning: [cppcoreguidelines-pro-type-vararg]
do not call c-style vararg functions
printf("Hello world!\n"); ^ -
docs/examples/demo/demo.hpp:6:11: warning: [modernize-use-default-member-init]
use default member initializer for 'useless'
char* useless; ^ {"\0"} -
docs/examples/demo/demo.hpp:7:9: warning: [modernize-use-default-member-init]
use default member initializer for 'numb'
int numb; ^ {0} -
docs/examples/demo/demo.hpp:11:11: warning: [modernize-use-trailing-return-type]
use a trailing return type for this function
void *not_useful(char *str){useless = str;} ~~~~~~^ auto -> void *
Have any feedback or feature suggestions? Share it here.
I don't intend to get this out in the next release. I still want to test this in the test repo to be sure there is no undesired behavior or side effects.
Totally agree. As more people use it, any undesired behavior or side effects will block user work.