Skip to content

Conversation

@JimunLee
Copy link
Contributor

Dear experts,

When I was testing to run the previous code at Hyperloop, I realized that it's not the code issue.
So, it should return the original code.

Thank you
Best regards,
Jimun

@github-actions
Copy link

github-actions bot commented Oct 15, 2025

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

@vkucera
Copy link
Collaborator

vkucera commented Oct 15, 2025

@JimunLee Please remove the <iostream> include.

@vkucera vkucera closed this Oct 15, 2025
@vkucera vkucera reopened this Oct 15, 2025
@JimunLee
Copy link
Contributor Author

@JimunLee Please remove the <iostream> include.

Thank you for the suggestion, but if I remove the include, It appears the issue at o2 MegaLinter issue.
So, I think, I'm fixing the source code to run well. After this process, I can fix the issues totally.

Thanks again,
Best regards,
Jimun

@smaff92 smaff92 merged commit 4abec76 into AliceO2Group:master Oct 16, 2025
20 of 22 checks passed
@smaff92
Copy link
Collaborator

smaff92 commented Oct 16, 2025

Merged for now, will make sure that she converts from iostream to O2logging for the next push.

ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
Co-authored-by: jimun_lee <jimun.lee@cern.ch>
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