Skip to content

Apply CI-standard CMake formatting to unblock ci-checks.sh#7884

Closed
Copilot wants to merge 2 commits into
copilot/remove-cchost-mentionsfrom
copilot/sub-pr-7883
Closed

Apply CI-standard CMake formatting to unblock ci-checks.sh#7884
Copilot wants to merge 2 commits into
copilot/remove-cchost-mentionsfrom
copilot/sub-pr-7883

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 14, 2026

This PR addresses the reported CI failure in scripts/ci-checks.sh, where the CMake format check failed on the current branch. The fix is limited to formatting CMakeLists.txt using the same formatter path used in CI.

  • CMake formatting alignment

    • Reflowed the file(READ ...) invocation in top-level CMakeLists.txt to match gersemi output.
    • No functional build logic changes; only formatting normalization to satisfy CI.
  • Scope

    • Single-file change: CMakeLists.txt
    • No behavior changes to runtime, tests, samples, or schema handling.
# Before
file(
  READ ${CCF_DIR}/doc/host_config_schema/host_config.json
  HOST_CONFIG_SCHEMA
)

# After
file(READ ${CCF_DIR}/doc/host_config_schema/host_config.json HOST_CONFIG_SCHEMA)

@maxtropets
Copy link
Copy Markdown
Collaborator

@copilot this PR has to be merged to main separately rather than into 7883

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 14, 2026

@maxtropets I've opened a new pull request, #7885, to work on those changes. Once the pull request is ready, I'll request review from you.

Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/fb2c3354-e196-424f-8e7a-9d52825c3668

Co-authored-by: maxtropets <16566519+maxtropets@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove stale cchost mentions across tests, samples, schema, and host config Apply CI-standard CMake formatting to unblock ci-checks.sh May 14, 2026
Copilot AI requested a review from maxtropets May 14, 2026 13:02
@maxtropets maxtropets closed this May 14, 2026
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.

2 participants