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

Refactor#83

Merged
Prayag2 merged 8 commits intomasterfrom
dev/clang-tidy
Dec 5, 2025
Merged

Refactor#83
Prayag2 merged 8 commits intomasterfrom
dev/clang-tidy

Conversation

@Prayag2
Copy link
Copy Markdown
Owner

@Prayag2 Prayag2 commented Dec 5, 2025

This PR refactors the entire codebase to adhere to C++ Core Guidelines. Some checks have been disabled:

  1. cppcoreguidelines-owning-memory: Qt allows the use of raw pointers due to its internal object management system, however, these still trigger this check.
  2. cppcoreguidelines-avoid-magic-numbers: This is only a temporary addition. When I get the time, I'll fix all violations to this check.

@Prayag2 Prayag2 merged commit 3b8addd into master Dec 5, 2025
0 of 3 checks passed
@github-project-automation github-project-automation Bot moved this from Available Tasks to Done in Drawy - Current Status Dec 5, 2025
@Prayag2 Prayag2 deleted the dev/clang-tidy branch December 5, 2025 16:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant