Skip to content

merge queue: checking main (92cccdf) and [#2098 + #2095 + #2096 + #2093 + #2092] together - #2099

Closed
mergify[bot] wants to merge 12 commits into
mainfrom
mergify/merge-queue/a4555fee43
Closed

merge queue: checking main (92cccdf) and [#2098 + #2095 + #2096 + #2093 + #2092] together#2099
mergify[bot] wants to merge 12 commits into
mainfrom
mergify/merge-queue/a4555fee43

Conversation

@mergify

@mergify mergify Bot commented Aug 18, 2026

Copy link
Copy Markdown

🎉 This combination of pull requests has been checked successfully and will be merged soon. 🎉

Branch main (92cccdf) and [#2098 + #2095 + #2096 + #2093 + #2092] are queued together for merge.

This pull request has been created by Mergify to check the mergeability of [#2098 + #2095 + #2096 + #2093 + #2092].
You don't need to do anything. Mergify will close this pull request automatically when it is complete.

Required conditions of queue rule main queue for merge:

  • check-success = all

Required conditions to stay in the queue:

---
checking_base_sha: 92cccdfc5b35221eedcf999166a2c97a1ccd1e11
previous_check_retries: []
previous_failed_batches: []
pull_requests:
  - number: 2098
    scopes: []
  - number: 2095
    scopes: []
  - number: 2096
    scopes: []
  - number: 2093
    scopes: []
  - number: 2092
    scopes: []
scopes: []
...

tdavidcl and others added 12 commits August 18, 2026 09:34
Include YAML workflow and config files in the loc metric and Plotly chart.
Commit-msg hooks rewrite the author and inject Co-authored-by. After
every git commit, amend with --no-verify so the initiating human is
Author and the trailer is Assisted-by: <agent> with no model name.

Assisted-by: Cursor Agent
The comparison loop used an uninitialized loop counter (`size_t i;`
instead of `size_t i = 0;`), so its starting value was whatever
garbage was left on the stack. Depending on compiler/optimization
level, this could skip comparisons entirely (silently passing a bad
fit) or read out of bounds, explaining the flaky/non-deterministic CI
failures. Also fixed a related out-of-bounds read: `ex_res` only had
3 entries but was indexed up to 3 (missing the expected R^2 value).

Assisted-by: Claude
@github-actions

Copy link
Copy Markdown
Contributor

Thanks @mergify[bot] for opening this PR!

You can do multiple things directly here:
1 - Comment pre-commit.ci run to run pre-commit checks.
2 - Comment pre-commit.ci autofix to apply fixes.
3 - Add label autofix.ci to fix authorship & pre-commit for every commit made.
4 - Add label full-ci to run the full test suite (default is light CI; full CI also runs on Mergify merge-queue branches).
5 - Add label trigger-ci to create an empty commit to trigger the CI.

Once the workflow completes a message will appear displaying informations related to the run.

Also the PR gets automatically reviewed by gemini, you can:
1 - Comment /gemini review to trigger a review
2 - Comment /gemini summary for a summary
3 - Tag it using @gemini-code-assist either in the PR or in review comments on files

@github-actions

Copy link
Copy Markdown
Contributor

Workflow report

workflow report corresponding to commit 7ec7cd9
Commiter email is 37929162+mergify[bot]@users.noreply.github.com
You are using github private e-mail. This prevent proper tracing of who contributed what, please disable it (see Keep my email addresses private).
GitHub page artifact URL GitHub page artifact link (can expire)

Pre-commit check report

Pre-commit check: ✅

trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check for merge conflicts................................................Passed
check that executables have shebangs.....................................Passed
check that scripts with shebangs are executable..........................Passed
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check for broken symlinks................................................Passed
check yaml...............................................................Passed
detect private key.......................................................Passed
No-tabs checker..........................................................Passed
Tabs remover.............................................................Passed
cmake-format.............................................................Passed
Validate GitHub Workflows................................................Passed
clang-format.............................................................Passed
ruff check...............................................................Passed
ruff format..............................................................Passed
Check doxygen headers....................................................Passed
Check license headers....................................................Passed
Check #pragma once.......................................................Passed
Check SYCL #include......................................................Passed
No ssh in git submodules remote..........................................Passed
No UTF-8 in files (except for authors)...................................Passed

Test pipeline can run.

Clang-tidy diff report


595 warnings generated.
Suppressed 596 warnings (595 in non-user code, 1 NOLINT).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.

631 warnings generated.
Suppressed 632 warnings (631 in non-user code, 1 NOLINT).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.

1217 warnings generated.
Suppressed 1218 warnings (1217 in non-user code, 1 NOLINT).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.

Doxygen diff with main

Removed warnings : 9
New warnings : 9
Warnings count : 8183 → 8183 (0.0%)

Detailed changes :
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:37: warning: Member Tscal (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:38: warning: Member dim (variable) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:39: warning: Member Kernel (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:41: warning: Member Config (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:41: warning: Member Tscal (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:42: warning: Member Storage (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:42: warning: Member dim (variable) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:43: warning: Member Kernel (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:44: warning: Member context (variable) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:45: warning: Member Config (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:45: warning: Member solver_config (variable) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:46: warning: Member Storage (typedef) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:46: warning: Member storage (variable) of class shammodels::sph::modules::ComputeEos is not documented.
- src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:48: warning: Member ComputeEos(ShamrockCtx &context, Config &solver_config, Storage &storage) (function) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:48: warning: Member context (variable) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:49: warning: Member solver_config (variable) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:50: warning: Member storage (variable) of class shammodels::sph::modules::ComputeEos is not documented.
+ src/shammodels/sph/include/shammodels/sph/modules/ComputeEos.hpp:52: warning: Member ComputeEos(ShamrockCtx &context, Config &solver_config, Storage &storage) (function) of class shammodels::sph::modules::ComputeEos is not documented.

@mergify mergify Bot closed this Aug 18, 2026
@mergify
mergify Bot deleted the mergify/merge-queue/a4555fee43 branch August 18, 2026 17:16
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.

3 participants