Skip to content

[PWGLF] updated with SOR time - #17891

Merged
prottayCMT merged 1 commit into
AliceO2Group:masterfrom
prottayCMT:master
Sep 12, 2026
Merged

prottayCMT merged 1 commit into
AliceO2Group:masterfrom
prottayCMT:master

Conversation

@prottayCMT

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title updated with SOR time [PWGLF] updated with SOR time Sep 12, 2026
@prottayCMT
prottayCMT enabled auto-merge (squash) September 12, 2026 08:11
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 9b124fd at 2026-09-12 12:44:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 5 errors and 9 warnings.'
Found 5 errors and 9 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/a635d8c3df28c76154c13f73ca4746905ed930e2/O2Physics-code-check/errors.txt
PWGLF/TableProducer/Common/zdcvector.cxx:49:1: error: included header unordered_map is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/TableProducer/Common/zdcvector.cxx:453:9: error: Value stored to 'qxZDCA' is never read [clang-analyzer-deadcode.DeadStores,-warnings-as-errors]
PWGLF/TableProducer/Common/zdcvector.cxx:454:9: error: Value stored to 'qxZDCC' is never read [clang-analyzer-deadcode.DeadStores,-warnings-as-errors]
PWGLF/TableProducer/Common/zdcvector.cxx:455:9: error: Value stored to 'qyZDCA' is never read [clang-analyzer-deadcode.DeadStores,-warnings-as-errors]
PWGLF/TableProducer/Common/zdcvector.cxx:456:9: error: Value stored to 'qyZDCC' is never read [clang-analyzer-deadcode.DeadStores,-warnings-as-errors]
++ [[ 9 -gt 0 ]]
++ cat /sw/BUILD/a635d8c3df28c76154c13f73ca4746905ed930e2/O2Physics-code-check/warnings.txt
PWGLF/TableProducer/Common/zdcvector.cxx:60:8: warning: constructor does not initialize these fields: ccdb [cppcoreguidelines-pro-type-member-init]
PWGLF/TableProducer/Common/zdcvector.cxx:375:17: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
PWGLF/TableProducer/Common/zdcvector.cxx:376:17: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
PWGLF/TableProducer/Common/zdcvector.cxx:399:13: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGLF/TableProducer/Common/zdcvector.cxx:416:13: warning: do not use 'else' after 'return' [readability-else-after-return]
PWGLF/TableProducer/Common/zdcvector.cxx:435:24: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Common/zdcvector.cxx:437:13: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Common/zdcvector.cxx:444:24: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/TableProducer/Common/zdcvector.cxx:446:13: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 5 -gt 0 ]]
++ exit 1
--

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 9b124fd at 2026-09-12 12:52:

## sw/BUILD/O2Physics-latest/log
c++: fatal error: Killed signal terminated program cc1plus
ninja: build stopped: subcommand failed.

Full log here.

@prottayCMT
prottayCMT merged commit e867377 into AliceO2Group:master Sep 12, 2026
13 of 15 checks passed
@vkucera

vkucera commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Dear @prottayCMT @omvazque , you have merged a PR adding errors. This is not allowed. Please fix them asap.

@vkucera

vkucera commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

@prottayCMT Do not enable auto-merge right after opening the PR.

  • It prevents the tag prefix to be part of the merged commit.
  • It indicates that you don't care about the results of the CI tests.

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.

4 participants