Skip to content

[PWGLF] kstar histogram now filling, solved several linter issues - #17222

Merged
maciacco merged 2 commits into
AliceO2Group:masterfrom
GiorgioAlbertoLucia:li4
Jul 27, 2026
Merged

[PWGLF] kstar histogram now filling, solved several linter issues#17222
maciacco merged 2 commits into
AliceO2Group:masterfrom
GiorgioAlbertoLucia:li4

Conversation

@GiorgioAlbertoLucia

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the pwglf label Jul 27, 2026
@github-actions

Copy link
Copy Markdown

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

@maciacco
maciacco enabled auto-merge (squash) July 27, 2026 14:19
@maciacco
maciacco merged commit 77adbd6 into AliceO2Group:master Jul 27, 2026
13 of 15 checks passed
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for c8e1593 at 2026-07-27 16:29:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 5 errors and 1 warnings.'
Found 5 errors and 1 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/956f62cc86cca7781ab5cd626eea150d26c8d908/O2Physics-code-check/errors.txt
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:251:8: error: constructor does not initialize these fields: mCcdb [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:288:85: error: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:572:14: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:624:14: error: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr,-warnings-as-errors]
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:676:7: error: do not use 'else' after 'return' [readability-else-after-return,-warnings-as-errors]
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/956f62cc86cca7781ab5cd626eea150d26c8d908/O2Physics-code-check/warnings.txt
PWGLF/TableProducer/Nuspex/he3HadronFemto.cxx:1311:58: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 5 -gt 0 ]]
++ exit 1
--

Full log here.

@GiorgioAlbertoLucia
GiorgioAlbertoLucia deleted the li4 branch July 27, 2026 14:49
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.

3 participants