Skip to content

Changed CI ctest verbose setting#6136

Merged
lrknox merged 8 commits into
HDFGroup:developfrom
brtnfld:cmake_verbose
Dec 30, 2025
Merged

Changed CI ctest verbose setting#6136
lrknox merged 8 commits into
HDFGroup:developfrom
brtnfld:cmake_verbose

Conversation

@brtnfld

@brtnfld brtnfld commented Dec 30, 2025

Copy link
Copy Markdown
Collaborator

Switched from always verbose ctest output to ONLY on test failures. Otherwise, it produces too much output when debugging test failures.


Important

Modify CI workflows to enable verbose ctest output only on test failures, reducing log verbosity during successful runs.

  • Behavior:
    • Set CTEST_OUTPUT_ON_FAILURE: 1 in environment to enable verbose ctest output only on test failures in analysis.yml, aocc.yml, and arm-main.yml.
    • Remove -V and -VV flags from ctest commands in analysis.yml, aocc.yml, and arm-main.yml to reduce verbosity during successful test runs.
  • Misc:
    • Consistent application of changes across 30+ workflow files to ensure uniform test output behavior.

This description was created by Ellipsis for 111e3a5. You can customize this summary. It will automatically update as commits are pushed.

hyoklee
hyoklee previously approved these changes Dec 30, 2025
@lrknox lrknox merged commit 21d9168 into HDFGroup:develop Dec 30, 2025
131 checks passed
@github-project-automation github-project-automation Bot moved this from To be triaged to Done in HDF5 - TRIAGE & TRACK Dec 30, 2025
@brtnfld brtnfld deleted the cmake_verbose branch December 30, 2025 20:04
brtnfld added a commit to brtnfld/hdf5 that referenced this pull request Mar 18, 2026
* switched from always verbose ctest output to ONLY on test failures

* Windows and macOS builds will skip clang-format check before building documentation

* resolved Doxygen warnings by adding H5G_link_t=H5L_type_t to the DOXYGEN_PREDEFINED list, by adding the macro definition, Doxygen now knows to treat H5G_link_t as H5L_type_
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants