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.2 in the pip group across 1 directory #246

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.2

Release notes

Sourced from cpp-linter's releases.

v1.9.2

πŸ› Bug fixes

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

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.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.

shenxianpeng commented 3 months ago

@dependabot close

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

2bndy5 commented 3 months ago

This supersedes #240

I also cherry picked the update to action.yml

@shenxianpeng It would be easier to cherry pick non-bot commits from a dependabot branch if the branch is not squashed (avoid force-pushing merge commits).

shenxianpeng commented 3 months ago

I expect to supersede this PR and keep #240 open, because on #240 we have comments and non-bot commits there.

But Dependabot does not support bumping version on #240, I should bump it manually but I forgot to do it later :)

2bndy5 commented 3 months ago

I'm ok with doing it this way if there's a link to the PR with the relevant thread comments.

dependabot[bot] commented 3 months ago

Looks like cpp-linter is updatable in another way, so this is no longer needed.