From 3f6ed0fc574bf63e530f541aa16db79733a27802 Mon Sep 17 00:00:00 2001 From: Patrick Shriwise Date: Fri, 20 Feb 2026 10:06:34 -0600 Subject: [PATCH 1/5] Update format check to provide suggestions if the a PR label is present. --- .github/workflows/format-check.yml | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/.github/workflows/format-check.yml b/.github/workflows/format-check.yml index cef14ca2c8c..68c37c8083f 100644 --- a/.github/workflows/format-check.yml +++ b/.github/workflows/format-check.yml @@ -5,6 +5,12 @@ on: workflow_dispatch: pull_request: + types: + - opened + - synchronize + - reopened + - labeled + - unlabeled branches: - develop - master @@ -12,6 +18,9 @@ on: jobs: cpp-linter: runs-on: ubuntu-latest + permissions: + contents: read + pull-requests: write steps: - uses: actions/checkout@v4 - uses: cpp-linter/cpp-linter-action@v2 @@ -23,10 +32,16 @@ jobs: files-changed-only: true tidy-checks: '-*' version: '15' # clang-format version + format-review: ${{ github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'cpp-format-suggest') }} + passive-reviews: ${{ github.event_name == 'pull_request' && contains(github.event.pull_request.labels.*.name, 'cpp-format-suggest') }} file-annotations: true step-summary: true extensions: 'cpp,h' - name: Failure Check if: steps.linter.outputs.checks-failed > 0 - run: echo "Some files failed the formatting check! See job summary and file annotations for more info" && exit 1 + run: | + echo "Some files failed the formatting check." + echo "See job summary and file annotations for details." + echo "To request inline suggestions on the next PR update, add the 'cpp-format-suggest' label to this pull request." + exit 1 From 249f940b5444e6c0e018f94ba7723fc6729dd88e Mon Sep 17 00:00:00 2001 From: Patrick Shriwise Date: Thu, 19 Feb 2026 23:17:43 -0600 Subject: [PATCH 2/5] Break C++ style --- src/plot.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plot.cpp b/src/plot.cpp index ea0555202d5..9d450d20853 100644 --- a/src/plot.cpp +++ b/src/plot.cpp @@ -2539,7 +2539,7 @@ extern "C" int openmc_solidraytrace_plot_set_diffuse_fraction( return OPENMC_E_INVALID_ARGUMENT; } - plt->diffuse_fraction() = diffuse_fraction; + plt->diffuse_fraction() = diffuse_fraction; return 0; } From f4e3b6240016db4951003d6c61066c4de818aab3 Mon Sep 17 00:00:00 2001 From: Patrick Shriwise Date: Fri, 20 Feb 2026 12:56:36 -0600 Subject: [PATCH 3/5] Empty commit for format suggestions From e2be78d488045054a751a239fd25ff3a7fbb72bb Mon Sep 17 00:00:00 2001 From: Patrick Shriwise Date: Fri, 20 Feb 2026 13:08:17 -0600 Subject: [PATCH 4/5] Trigger C++ check From cff99b902806785b097764a694b83c228a5dcc2a Mon Sep 17 00:00:00 2001 From: Patrick Shriwise Date: Fri, 20 Feb 2026 13:17:26 -0600 Subject: [PATCH 5/5] Another trigger