Skip to content

[PWGLF] Fix place of histogram filling - #17279

Merged
gbencedi merged 1 commit into
AliceO2Group:masterfrom
gbencedi:master
Jul 31, 2026
Merged

[PWGLF] Fix place of histogram filling#17279
gbencedi merged 1 commit into
AliceO2Group:masterfrom
gbencedi:master

Conversation

@gbencedi

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Fix place of histogram filling [PWGLF] Fix place of histogram filling Jul 31, 2026
@gbencedi
gbencedi enabled auto-merge (squash) July 31, 2026 19:56

@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 @gbencedi.

@gbencedi
gbencedi merged commit 0f56e6d into AliceO2Group:master Jul 31, 2026
14 of 15 checks passed
@alibuild

alibuild commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for c605e70 at 2026-08-01 10:08:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 3 errors and 0 warnings.'
Found 3 errors and 0 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/2da66c14b6d2f24fd5f19d79f2c1c2ecf2cfbfde/O2Physics-code-check/errors.txt
PWGLF/Tasks/GlobalEventProperties/flattenictyPikp.cxx:1902:45: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
PWGLF/Tasks/GlobalEventProperties/flattenictyPikp.cxx:1908:45: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
PWGLF/Tasks/GlobalEventProperties/flattenictyPikp.cxx:1914:46: error: repeated branch body in conditional chain [bugprone-branch-clone,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

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