search

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
91 stars 21 forks source link

Bump cpp-linter from 1.8.1 to 1.9.1 in the pip group across 1 directory #240

Closed dependabot[bot] closed 3 months ago

dependabot[bot] commented 4 months ago

Bumps the pip group with 1 update in the / directory: cpp-linter.

Updates cpp-linter from 1.8.1 to 1.9.1

Release notes

Sourced from cpp-linter's releases.

v1.9.1

🐛 Bug fixes

  • prevent dead links (to tidy diagnostics pages) in thread comments & step summaries by @​2bndy5 in #106

Full Changelog: https://github.com/cpp-linter/cpp-linter/compare/v1.9.0...v1.9.1

v1.9.0

🚀 New features and improvements

📝 Documentation updates

Full Changelog: https://github.com/cpp-linter/cpp-linter/compare/v1.8.1...v1.9.0

Commits


Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore major version` will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself) - `@dependabot ignore minor version` will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself) - `@dependabot ignore ` will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself) - `@dependabot unignore ` will remove all of the ignore conditions of the specified dependency - `@dependabot unignore ` will remove the ignore condition of the specified dependency and ignore conditions
github-actions[bot] commented 4 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.hpp - docs/examples/demo/demo.cpp
clang-tidy reports: 7 concern(s) - **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 * ``` - **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"); ^ ```

Have any feedback or feature suggestions? Share it here.

2bndy5 commented 4 months ago

I just have a question about testing here https://github.com/cpp-linter/test-cpp-linter-action/pull/34#issuecomment-2097437550

I still need to explore idea from this comment

2bndy5 commented 4 months ago

After testing the new input options (see this thread comment), I'm confident about these improvements.

We need only focus on #237 for now

dependabot[bot] commented 3 months ago

This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests.

To ignore these dependencies, configure ignore rules in dependabot.yml