CESNET / ipfixprobe

BSD 3-Clause "New" or "Revised" License
40 stars 18 forks source link

Draft: Clang format - code reformatting #192

Open SiskaPavel opened 1 year ago

SiskaPavel commented 1 year ago

Adds new options to the Makefile: Introduces new options, "format-check" and "format-fix," to the Makefile for checking and fixing code formatting in .cpp and .hpp files.

Performs code reformatting across the entire repository: Aims to achieve consistent and uniform code formatting by addressing code indentation, spacing, and other style-related issues in all relevant code files.

Incorporates a new step into the Continuous Integration (CI) process: Enhances code quality by adding the "make format-check" command to the CI pipeline, ensuring that code formatting is validated automatically for all code changes pushed to the repository. This step is crucial for maintaining code consistency and adherence to coding standards.

codecov-commenter commented 1 year ago

Codecov Report

Attention: 2300 lines in your changes are missing coverage. Please review.

Comparison is base (cfc02c2) 41.26% compared to head (336c934) 41.52%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #192 +/- ## ========================================== + Coverage 41.26% 41.52% +0.25% ========================================== Files 94 94 Lines 8665 9152 +487 Branches 1299 1300 +1 ========================================== + Hits 3576 3800 +224 - Misses 4321 4572 +251 - Partials 768 780 +12 ``` | [Flag](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET) | Coverage Δ | | |---|---|---| | [tests](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET) | `41.52% <35.04%> (+0.25%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET) | Coverage Δ | | |---|---|---| | [include/ipfixprobe/input.hpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-aW5jbHVkZS9pcGZpeHByb2JlL2lucHV0LmhwcA==) | `100.00% <100.00%> (ø)` | | | [include/ipfixprobe/output.hpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-aW5jbHVkZS9pcGZpeHByb2JlL291dHB1dC5ocHA=) | `100.00% <100.00%> (ø)` | | | [process/http.cpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9odHRwLmNwcA==) | `44.12% <ø> (ø)` | | | [process/http.hpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9odHRwLmhwcA==) | `32.05% <ø> (+0.40%)` | :arrow_up: | | [process/icmp.cpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9pY21wLmNwcA==) | `28.57% <ø> (+1.90%)` | :arrow_up: | | [process/icmp.hpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9pY21wLmhwcA==) | `0.00% <ø> (ø)` | | | [process/idpcontent.cpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9pZHBjb250ZW50LmNwcA==) | `96.15% <ø> (+0.32%)` | :arrow_up: | | [process/idpcontent.hpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9pZHBjb250ZW50LmhwcA==) | `35.13% <ø> (+6.10%)` | :arrow_up: | | [process/md5.cpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9tZDUuY3Bw) | `86.16% <ø> (ø)` | | | [process/mpls.cpp](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET#diff-cHJvY2Vzcy9tcGxzLmNwcA==) | `30.76% <ø> (ø)` | | | ... and [76 more](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET) | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/CESNET/ipfixprobe/pull/192/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CESNET)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.