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

ci(pre-commit): autoupdate #174

Merged
merged 3 commits into from
Feb 4, 2025
Merged

ci(pre-commit): autoupdate #174

merged 3 commits into from
Feb 4, 2025

Conversation

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from edd3291 to e4ffc9f Compare July 8, 2024 17:18
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 02ae5bc to 0dee83d Compare August 5, 2024 17:36
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 0b2af6a to 9c7f011 Compare August 26, 2024 17:30
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d9bd4db to eb44ec3 Compare September 30, 2024 17:29
Copy link

Copy link

codecov bot commented Sep 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.24%. Comparing base (f4da047) to head (427d031).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #174   +/-   ##
=======================================
  Coverage   27.24%   27.24%           
=======================================
  Files         104      104           
  Lines        9445     9447    +2     
  Branches     2324     2326    +2     
=======================================
+ Hits         2573     2574    +1     
+ Misses       6436     6435    -1     
- Partials      436      438    +2     
Flag Coverage Δ
differential 27.24% <100.00%> (?)
total ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from de4e0ed to fa1d0db Compare October 14, 2024 17:34
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 1a0c86f to 074abef Compare October 21, 2024 17:30
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from a30074b to 3e852ce Compare November 4, 2024 17:34
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d76c00d to 770ee6c Compare November 25, 2024 17:26
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 522c121 to d0cc79d Compare December 2, 2024 17:33
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 131262e to fd3218f Compare December 16, 2024 17:28
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 2a2f24e to de9e999 Compare January 6, 2025 17:34
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from d0e3f95 to f0fab9a Compare January 20, 2025 17:19
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 1c7ead6 to 8e602f7 Compare January 27, 2025 17:26
updates:
- [github.com/pre-commit/pre-commit-hooks: v4.6.0 → v5.0.0](pre-commit/pre-commit-hooks@v4.6.0...v5.0.0)
- [github.com/igorshubovych/markdownlint-cli: v0.40.0 → v0.44.0](igorshubovych/markdownlint-cli@v0.40.0...v0.44.0)
- [github.com/pre-commit/mirrors-prettier: v3.1.0 → v4.0.0-alpha.8](pre-commit/mirrors-prettier@v3.1.0...v4.0.0-alpha.8)
- [github.com/tier4/pre-commit-hooks-ros: v0.8.0 → v0.10.0](tier4/pre-commit-hooks-ros@v0.8.0...v0.10.0)
- [github.com/scop/pre-commit-shfmt: v3.8.0-1 → v3.10.0-2](scop/pre-commit-shfmt@v3.8.0-1...v3.10.0-2)
- [github.com/pycqa/isort: 5.13.2 → 6.0.0](PyCQA/isort@5.13.2...6.0.0)
- [github.com/psf/black: 24.4.2 → 25.1.0](psf/black@24.4.2...25.1.0)
- [github.com/PyCQA/flake8: 7.0.0 → 7.1.1](PyCQA/flake8@7.0.0...7.1.1)
- [github.com/pre-commit/mirrors-clang-format: v18.1.5 → v19.1.7](pre-commit/mirrors-clang-format@v18.1.5...v19.1.7)
- [github.com/cpplint/cpplint: 1.6.1 → 2.0.0](cpplint/cpplint@1.6.1...2.0.0)
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 993692d to 750ddfe Compare February 3, 2025 17:33
pre-commit-ci bot and others added 2 commits February 3, 2025 17:35
…t-you-use rule

Signed-off-by: Max SCHMELLER <max.schmeller@tier4.jp>
@mojomex mojomex self-requested a review February 4, 2025 00:02
Copy link
Collaborator

@mojomex mojomex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • confirmed that pre-commit is installable and runnable
  • tweaked cpplint settings to be less pedantic about the <filesystem> header and about C-style casts
  • applied the suggested include-what-you-use fixes from cpplint
  • built and ran unit tests
  • launched (without sensor) each vendor wrapper
  • manually confirmed applied import fixes

@mojomex mojomex merged commit 86fe233 into main Feb 4, 2025
11 of 12 checks passed
@mojomex mojomex deleted the pre-commit-ci-update-config branch February 4, 2025 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant