Skip to content

Conversation

@dariopellegrino00
Copy link
Collaborator

@dariopellegrino00 dariopellegrino00 commented Aug 20, 2025

Summary

  • Fix old missed clang-tidy check causing failure
  • Fix strcpy urgent warning
  • Fix cppcheck checking all dependencies
  • Fix cppcheck not failing on urgent warnings
  • Add cppcheck print style issues after analysis success

Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
@dariopellegrino00 dariopellegrino00 self-assigned this Aug 20, 2025
Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
@dariopellegrino00 dariopellegrino00 force-pushed the compile-errors-fix branch 2 times, most recently from 52d7b58 to 58e6372 Compare August 21, 2025 09:43
@dariopellegrino00 dariopellegrino00 changed the title Fix missed old clang-tidy failing Fix and Refactor CI code and build checks Aug 21, 2025
Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
@dariopellegrino00 dariopellegrino00 changed the title Fix and Refactor CI code and build checks CI: Fix and Refactor code and build checks Aug 21, 2025
@dariopellegrino00 dariopellegrino00 merged commit 8b93611 into develop Aug 21, 2025
1 check passed
@dariopellegrino00 dariopellegrino00 deleted the compile-errors-fix branch August 21, 2025 10:00
dariopellegrino00 added a commit that referenced this pull request Aug 21, 2025
* Fix and Refactor code and CI  (#7)

Changes included:
* Add CI workflows for build and static analysis
* Fix critical warning
* Minor performance improvements


Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
dariopellegrino00 added a commit that referenced this pull request Aug 21, 2025
Changes included:
* Add CI workflows for build and static analysis
* Fix critical warning
* Minor performance improvements

Signed-off-by: Dario Pellegrino <dario.pellegrino@voismart.it>
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