Skip to content
New issue

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

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

Already on GitHub? # to your account

[CPU] Fix modernize-loop-convert clang-tidy remarks #28966

Merged
merged 11 commits into from
Feb 26, 2025

Conversation

aobolensk
Copy link
Contributor

@aobolensk aobolensk commented Feb 13, 2025

Details:

  • Fix "modernize-loop-convert" remarks reported by clang-tidy
  • Enable "modernize-loop-convert" clang-tidy checks on CI by default

Tickets:

  • N/A

@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Feb 13, 2025
@aobolensk aobolensk marked this pull request as ready for review February 13, 2025 06:56
@aobolensk aobolensk requested review from a team as code owners February 13, 2025 06:56
@aobolensk aobolensk force-pushed the modernize-loop branch 6 times, most recently from fecc0b1 to 8605371 Compare February 14, 2025 07:47
@aobolensk aobolensk force-pushed the modernize-loop branch 2 times, most recently from 4dbfb1d to 977c5fd Compare February 20, 2025 14:42
@EgorDuplensky
Copy link
Contributor

@aobolensk I think we better to update .clang-tidy in scope of this PR, even currently we are enabling the whole modernize-* scope in scope of another PR. Just to have correct state on master for every commit.

@aobolensk
Copy link
Contributor Author

@aobolensk I think we better to update .clang-tidy in scope of this PR, even currently we are enabling the whole modernize-* scope in scope of another PR. Just to have correct state on master for every commit.

Done

@aobolensk aobolensk force-pushed the modernize-loop branch 2 times, most recently from f037471 to 3dab1e2 Compare February 25, 2025 12:09
@dmitry-gorokhov dmitry-gorokhov added this pull request to the merge queue Feb 26, 2025
Merged via the queue into openvinotoolkit:master with commit 00dbc94 Feb 26, 2025
182 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
category: CPU OpenVINO CPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants