Skip to content

[PWGLF,PWGMM] Cleanup following reassociation checks - #18011

Merged
gbencedi merged 1 commit into
AliceO2Group:masterfrom
gbencedi:master
Sep 22, 2026
Merged

gbencedi merged 1 commit into
AliceO2Group:masterfrom
gbencedi:master

Conversation

@gbencedi

Copy link
Copy Markdown
Collaborator
  • Major cleanup following reassociation checks
  • Keep only processes for CentFT0C estimator

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Cleanup following reassociation checks [PWGLF,PWGMM] Cleanup following reassociation checks Sep 22, 2026
@gbencedi
gbencedi enabled auto-merge (squash) September 22, 2026 17:02

@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 0d3e70a into AliceO2Group:master Sep 22, 2026
14 of 15 checks passed
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for a109d86 at 2026-09-22 21:47:

## sw/BUILD/O2-latest/log
Error: missing ; at end of rule at /sw/SOURCES/O2/daily-20260922-0000/daily-20260922-0000/Detectors/TPC/base/src/TPCBaseLinkDef.h:75:56
Error: missing ; at end of rule at /sw/SOURCES/O2/daily-20260922-0000/daily-20260922-0000/Detectors/TPC/base/src/TPCBaseLinkDef.h:83:135
Error: missing ; at end of rule at /sw/SOURCES/O2/daily-20260922-0000/daily-20260922-0000/Detectors/TPC/base/src/TPCBaseLinkDef.h:84:135
Error: missing ; at end of rule at /sw/SOURCES/O2/daily-20260922-0000/daily-20260922-0000/Detectors/Upgrades/ALICE3/ECal/base/src/ECalBaseLinkDef.h:19:48
Error: missing ; at end of rule at /sw/SOURCES/O2/daily-20260922-0000/daily-20260922-0000/Detectors/Upgrades/ALICE3/RICH/base/src/RICHBaseLinkDef.h:18:48
Error: missing ; at end of rule at /sw/SOURCES/O2/daily-20260922-0000/daily-20260922-0000/Detectors/Upgrades/ALICE3/MID/base/src/MI3BaseLinkDef.h:18:47


## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 1 warnings.'
Found 1 errors and 1 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/56502446293609729ac446ac28a6732fc19b9977/O2Physics-code-check/errors.txt
PWGMM/Mult/Tasks/dndetaMFTPbPb.cxx:26:1: error: included header CollisionAssociationTables.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/56502446293609729ac446ac28a6732fc19b9977/O2Physics-code-check/warnings.txt
PWGMM/Mult/Tasks/dndetaMFTPbPb.cxx:2382:37: warning: boolean expression can be simplified by DeMorgan's theorem [readability-simplify-boolean-expr]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@vkucera

vkucera commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

@gbencedi Please remember to fix errors before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants