Skip to content

fix(ci): sort_include fails with multiline sight_add_target macros

Flavien BRIDAULT requested to merge 49-core-sort_include-fails-with-multiline-sight_add_target-macros into dev

Description

This fixes the sort of includes when targets use a multiline sight_add_target()

Closes #49 (closed)

How to test it?

Repeat the steps described in #49 (closed).

Additional fixes

I enriched the cmake-format configuration files to fix the parsing of sight_add_target and module_param. The positional arguments should now be processed as they should. Example:

# Before
sight_add_target(
    ui_qt
    TYPE
    LIBRARY
    PCH
    pchService
    WARNINGS_AS_ERRORS
    OFF
)

# After
sight_add_target(
    ui_qt
    TYPE LIBRARY
    PCH pchService
    WARNINGS_AS_ERRORS OFF
)
Edited by Flavien BRIDAULT

Merge request reports