-
Notifications
You must be signed in to change notification settings - Fork 130
Open
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededrefactoring
Description
Important
Refactoring changes should be rebased and merged against this branch:
feature/655-api-modernization-and-refactoring
Summary
Add a .clang-format file to the repository root to enforce consistent formatting across all C++ source files. This should be done after the naming refactoring is complete, so the format reflects the final coding style.
Requirements
The .clang-format should enforce:
- C++17 standard
- Consistent brace style (Allman or K&R — to be decided based on current codebase majority)
- 4-space indentation (matching current code)
- Column limit (120 or 140, matching existing style)
- Pointer/reference alignment:
const String & namestyle currently used - Include sorting and grouping
Deliverables
-
.clang-formatfile at repository root -
.clang-format-ignoreif any files should be excluded (e.g., generated code, third-party) - Verification: running
clang-formaton a sample of files produces reasonable output - Optional: CI step to check formatting
Depends on
- All naming convention changes should be finalized first
Acceptance criteria
-
.clang-formatcommitted to repository root - Format is consistent with the established code style
- No formatting conflicts with existing code patterns
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededrefactoring
Projects
Status
No status