Skip to content

[PWGDQ] Fix includes - #17789

Merged
mcoquet642 merged 1 commit into
AliceO2Group:masterfrom
vkucera:iwyu-PWGDQ
Sep 7, 2026
Merged

[PWGDQ] Fix includes#17789
mcoquet642 merged 1 commit into
AliceO2Group:masterfrom
vkucera:iwyu-PWGDQ

Conversation

@vkucera

@vkucera vkucera commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the pwgdq label Sep 5, 2026
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 1 errors, ⚠️ 63 warnings, 🔕 20 disabled

@github-actions github-actions Bot changed the title Fix includes [PWGDQ] Fix includes Sep 5, 2026
@vkucera
vkucera marked this pull request as ready for review September 6, 2026 14:29
@alibuild

alibuild commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 715c3b9 at 2026-09-06 23:24:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 2 errors and 2 warnings.'
Found 2 errors and 2 warnings.
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/5fab9e67f9e2e4807af7ae053cc1e879535dcdfe/O2Physics-code-check/errors.txt
PWGDQ/Macros/dqFlowAccWeights.C:49:10: error: variable 'file' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
PWGDQ/Macros/dqFlowAccWeights.C:56:15: error: variable 'weights' is not initialized [cppcoreguidelines-init-variables,-warnings-as-errors]
++ [[ 2 -gt 0 ]]
++ cat /sw/BUILD/5fab9e67f9e2e4807af7ae053cc1e879535dcdfe/O2Physics-code-check/warnings.txt
PWGDQ/Macros/dqFlowAccWeights.C:69:14: warning: redundant call to 'c_str' [readability-redundant-string-cstr]
PWGDQ/Tasks/mftMchMatcher.cxx:406:14: warning: redundant boolean literal in conditional return statement [readability-simplify-boolean-expr]
++ [[ 2 -gt 0 ]]
++ exit 1
--

Full log here.

@mcoquet642
mcoquet642 merged commit bf3bdb1 into AliceO2Group:master Sep 7, 2026
12 of 14 checks passed
@vkucera
vkucera deleted the iwyu-PWGDQ branch September 7, 2026 09:26
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