Skip to content

Apply clang-tidy on both headers and sources #206

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

Merged
merged 6 commits into from
Jan 13, 2025

Conversation

aobolensk
Copy link
Member

No description provided.

@aobolensk aobolensk force-pushed the clang-tidy-headers-apply branch from 64a8157 to 57592fb Compare January 13, 2025 16:56
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

Attention: Patch coverage is 95.78652% with 15 lines in your changes missing coverage. Please review.

Project coverage is 94.24%. Comparing base (a8c0499) to head (2fd3918).
Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
modules/core/task/src/task.cpp 82.75% 3 Missing and 2 partials ⚠️
tasks/mpi/example/src/ops_mpi.cpp 92.85% 0 Missing and 3 partials ⚠️
modules/core/perf/src/perf.cpp 90.00% 0 Missing and 2 partials ⚠️
tasks/omp/example/src/ops_omp.cpp 93.54% 0 Missing and 2 partials ⚠️
tasks/tbb/example/src/ops_tbb.cpp 93.75% 0 Missing and 2 partials ⚠️
tasks/stl/example/src/ops_stl.cpp 96.15% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #206      +/-   ##
==========================================
- Coverage   94.73%   94.24%   -0.50%     
==========================================
  Files          34       34              
  Lines         608      608              
  Branches      184      184              
==========================================
- Hits          576      573       -3     
- Misses         11       14       +3     
  Partials       21       21              

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

@allnes allnes merged commit beda560 into learning-process:master Jan 13, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants