-
Notifications
You must be signed in to change notification settings - Fork 614
[PWGLF] removed unnecessary disable directives for o2linter #11600
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
O2 linter results: ❌ 27 errors, |
|
Thanks @alcaliva |
|
Error while checking build/O2Physics/o2 for 649659d at 2025-06-13 22:51: Full log here. |
@hahassan7 This seems like a bug in |
|
Hi @vkucera, I agree, because that function cannot (and does not) modify the jet, so it should not take a non-const ref. |
Pull request was closed
|
I decided to close it because the new PR was "hiding" behind this one and I wanted it to be more visible. |
@vkucera, I fixed the issue you mentioned in the previous PR here