Skip to content

[PWGLF] Fix cppcheck warning in identified RAA task - #14081

Merged
vkucera merged 1 commit into
masterfrom
mp-branch-3
Dec 4, 2025
Merged

[PWGLF] Fix cppcheck warning in identified RAA task#14081
vkucera merged 1 commit into
masterfrom
mp-branch-3

Conversation

@mpuccio

@mpuccio mpuccio commented Dec 4, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Dec 4, 2025

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Fix cpplinter warning in identified RAA task [PWGLF] Fix cpplinter warning in identified RAA task Dec 4, 2025
@mpuccio
mpuccio enabled auto-merge (squash) December 4, 2025 08:52

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

@vkucera vkucera changed the title [PWGLF] Fix cpplinter warning in identified RAA task [PWGLF] Fix cppcheck warning in identified RAA task Dec 4, 2025
@vkucera
vkucera disabled auto-merge December 4, 2025 09:03
@vkucera
vkucera enabled auto-merge (squash) December 4, 2025 09:03
@vkucera

vkucera commented Dec 4, 2025

Copy link
Copy Markdown
Collaborator

Thanks @mpuccio for the fix. I just corrected the title, since the error is from cppcheck. cpplinter on the other hand complains about missing includes.

@vkucera
vkucera merged commit 204f5f5 into master Dec 4, 2025
14 of 17 checks passed
@vkucera
vkucera deleted the mp-branch-3 branch December 4, 2025 10:27
mapalhares pushed a commit to mapalhares/O2Physics that referenced this pull request Jan 20, 2026
fcolamar pushed a commit to fcolamar/O2Physics that referenced this pull request Feb 4, 2026
karlazertova pushed a commit to karlazertova/O2Physics that referenced this pull request Feb 5, 2026
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