Skip to content

add header include order checking to CI #269

@jacobmerson

Description

@jacobmerson

In PR #260 @hgangwar updates the header file handling to use FILE_SETS of type HEADER. His goal for that PR is to address the inconsistent header locations between installation and build modes. However, one additional advantage is that this allows us to ensure that all public headers do not depend on include order.

Once PR@260 is merged, this can simply be handled by adding -DCMAKE_VERIFY_INTERFACE_HEADER_SETS=TRUE to the build PCMS build configuration.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions