Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

Add CMake option for Cortex code quality assurance #2085

Merged
merged 2 commits into from
Mar 10, 2025
Merged

Add CMake option for Cortex code quality assurance #2085

merged 2 commits into from
Mar 10, 2025

Conversation

qnixsynapse
Copy link
Contributor

Describe Your Changes

  • This pull request introduces an option to enable ASan, debugging symbols and activate all compiler warnings, treating them as errors only for debugging purposes. This change aims to create a way to improve code quality by encouraging developers to address potential issues early.

Please note that this option is disabled by default. Currently, enabling it will break the build; however, I plan to resolve these issues over time. Once stabilized, we can create a CI workflow to catch potential errors and memory issues early.

@qnixsynapse qnixsynapse requested a review from vansangpfiev March 7, 2025 15:39
Copy link
Contributor

@vansangpfiev vansangpfiev left a comment

Choose a reason for hiding this comment

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

LGTM.
Let merge the PR and fix the warnings in separated PRs.

@qnixsynapse qnixsynapse merged commit 88efee0 into dev Mar 10, 2025
7 of 8 checks passed
@qnixsynapse qnixsynapse deleted the cqa branch March 10, 2025 08:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants