Skip to content

[PWGEM/Dilepton] fix partition#16961

Merged
dsekihat merged 1 commit into
AliceO2Group:masterfrom
dsekihat:master
Jul 8, 2026
Merged

[PWGEM/Dilepton] fix partition#16961
dsekihat merged 1 commit into
AliceO2Group:masterfrom
dsekihat:master

Conversation

@dsekihat

@dsekihat dsekihat commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

[PWGEM/Dilepton] fix partition

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 131 errors, ⚠️ 219 warnings, 🔕 0 disabled

@dsekihat dsekihat enabled auto-merge (squash) July 8, 2026 12:42

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @dsekihat.

@alibuild

alibuild commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for d6fe0a7 at 2026-07-08 15:25:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 26 errors and 1 warnings.'
Found 26 errors and 1 warnings.
++ [[ 26 -gt 0 ]]
++ cat /sw/BUILD/89ad2e3aa1bd1b545970dc400245a30da4499abb/O2Physics-code-check/errors.txt
PWGEM/Dilepton/Core/Dilepton.h:88:10: error: inclusion of deprecated C++ header 'math.h'; consider using 'cmath' instead [modernize-deprecated-headers,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:107:8: error: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:339:20: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:381:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:382:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:398:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:399:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:415:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:416:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:433:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:434:7: error: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:503:39: error: use nullptr [modernize-use-nullptr,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:504:43: error: use nullptr [modernize-use-nullptr,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:558:25: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:579:11: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:862:7: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:884:149: error: the const qualified parameter 'weightvector' is copied for each invocation; consider making it a reference [performance-unnecessary-value-param,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:888:9: error: if with identical then and else branches [bugprone-branch-clone,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:914:105: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1305:13: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1325:13: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1435:24: error: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1462:39: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1546:7: error: if with identical then and else branches [bugprone-branch-clone,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1564:105: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
PWGEM/Dilepton/Core/Dilepton.h:1585:13: error: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays,-warnings-as-errors]
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/89ad2e3aa1bd1b545970dc400245a30da4499abb/O2Physics-code-check/warnings.txt
PWGEM/Dilepton/Core/Dilepton.h:505:24: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 26 -gt 0 ]]
++ exit 1
--

Full log here.

@dsekihat dsekihat merged commit bd55b35 into AliceO2Group:master Jul 8, 2026
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants