Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump cpp-linter from 1.10.1 to 1.10.2 in the pip group #267

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Sep 20, 2024

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

Updates cpp-linter from 1.10.1 to 1.10.2

Release notes

Sourced from cpp-linter's releases.

v1.10.2

🐛 Bug fixes

  • swap file names in paginated changed files's diff by @​2bndy5 in #121

Full Changelog: cpp-linter/cpp-linter@v1.10.1...v1.10.2

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 <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> 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 <dependency name> 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 <dependency name> 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 <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

Bumps the pip group with 1 update: [cpp-linter](https://github.com/cpp-linter/cpp-linter).


Updates `cpp-linter` from 1.10.1 to 1.10.2
- [Release notes](https://github.com/cpp-linter/cpp-linter/releases)
- [Commits](cpp-linter/cpp-linter@v1.10.1...v1.10.2)

---
updated-dependencies:
- dependency-name: cpp-linter
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file patch A patch version bump python Pull requests that update Python code labels Sep 20, 2024
Copy link
Contributor

github-actions bot commented Sep 20, 2024

Cpp-Linter Report ⚠️

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)

Have any feedback or feature suggestions? Share it here.

@2bndy5 2bndy5 merged commit 7b41c60 into main Sep 20, 2024
65 checks passed
@2bndy5 2bndy5 deleted the dependabot/pip/pip-881906fd05 branch September 20, 2024 06:44
@shenxianpeng
Copy link
Collaborator

shenxianpeng commented Sep 20, 2024

I see some test errro on macos https://github.com/cpp-linter/cpp-linter-action/actions/runs/10954530716

not sure why. just bring our attention here.

image

@shenxianpeng
Copy link
Collaborator

shenxianpeng commented Sep 20, 2024

It should be brew does not support to install llvm 9, 10, 11 anymore. but we can install with clang-tools. it seems should be fine.

@2bndy5
Copy link
Collaborator

2bndy5 commented Sep 20, 2024

Yes, that's exactly it. When adding the homebrew solution, I couldn't find a efficient way of knowing exactly which formulas were yanked or deprecated. Trying to install those versions will yield a warning/error and a non-zero exit code. That's why I made the step infallible, so we can fallback to using clang-tools-pip.

continue-on-error: true

Typically, the versions affected are older and the static binaries of those version do not suffer from segmentation faults or large binary sizes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file patch A patch version bump python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants