Skip to content

Conversation

@f-michaut
Copy link
Contributor

  • Added clang-tidy config file + CI
  • Reformated the code to fit the new Style
  • Added SSL_Utils for better smart pointers of SSL objects
  • Added Certificate class + extras to manipulate Certificates

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch 2 times, most recently from 947388a to 666fa99 Compare August 5, 2025 03:10
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

There were too many comments to post at once. Showing the first 25 out of 27. Check the log or trigger a new build to see more.

@f-michaut f-michaut enabled auto-merge (squash) August 5, 2025 03:32
@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch 4 times, most recently from 524019e to e747afd Compare August 5, 2025 04:21
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from e747afd to 5e68e54 Compare August 5, 2025 04:38
@f-michaut f-michaut disabled auto-merge August 5, 2025 04:40
@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 5e68e54 to 1b3368a Compare August 5, 2025 04:47
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 1b3368a to 2280fa6 Compare August 5, 2025 05:00
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 2280fa6 to d0bc280 Compare August 5, 2025 13:46
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from d0bc280 to 58bc224 Compare August 5, 2025 14:11
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 58bc224 to e9e639e Compare August 5, 2025 14:43
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch 4 times, most recently from daeb5d3 to 9c88ad3 Compare August 5, 2025 15:09
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 9c88ad3 to 1b28669 Compare August 5, 2025 15:13
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 1b28669 to a7845b4 Compare August 5, 2025 15:15
@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from a7845b4 to 1866ee0 Compare August 5, 2025 15:18
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 1866ee0 to 36398f4 Compare August 5, 2025 15:37
- Added clang-tidy config file + CI
- Reformated the code to fit the new Style
- Added SSL_Utils for better smart pointers of SSL objects
- Added Certificate class + extras to manipulate Certificates
@f-michaut f-michaut force-pushed the styling-certificates-and-ssl-utils branch from 36398f4 to f745aca Compare August 5, 2025 16:08
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

clang-tidy review says "All clean, LGTM! 👍"

@f-michaut f-michaut merged commit 7e658cf into master Aug 5, 2025
13 checks passed
@f-michaut f-michaut deleted the styling-certificates-and-ssl-utils branch August 5, 2025 16:25
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.

2 participants