Skip to content

Fix Include Dependencies and Standardize Formatting#15

Merged
MitchellThompkins merged 10 commits intodevelopfrom
clang-format
Feb 19, 2026
Merged

Fix Include Dependencies and Standardize Formatting#15
MitchellThompkins merged 10 commits intodevelopfrom
clang-format

Conversation

@MitchellThompkins
Copy link
Owner

@MitchellThompkins MitchellThompkins commented Feb 19, 2026

Overview

This PR resolves build failures caused by alphabetical include reordering. It ensures all headers are self-contained and standardizes the project's code style.

Key Changes

  • Header Integrity: Fixed missing internal dependencies in math/complex.hpp, matrix/matrix.hpp, and all decomposition headers. They now explicitly include the math utilities they require.
  • Tooling:
    • Enabled SortIncludes: true in .clang-format.
    • Added make check-format to the Makefile.
    • Integrated a formatting check into the GitHub Actions CI pipeline.
  • Standardization: Applied a consistent C++17 formatting style across the entire codebase

@MitchellThompkins MitchellThompkins changed the base branch from main to develop February 19, 2026 07:53
@MitchellThompkins MitchellThompkins changed the title Clang format Fix Include Dependencies and Standardize Formatting Feb 19, 2026
@MitchellThompkins MitchellThompkins merged commit 86a1fe2 into develop Feb 19, 2026
4 checks passed
@MitchellThompkins MitchellThompkins deleted the clang-format branch February 19, 2026 08:35
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.

1 participant

Comments