Skip to content

Conversation

@ddobrigk
Copy link
Collaborator

No description provided.

@github-actions
Copy link

github-actions bot commented Sep 17, 2025

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

@ddobrigk
Copy link
Collaborator Author

ddobrigk commented Sep 17, 2025

Previously: addressed only cpplint. Moved to draft for now, as other warnings could be fixed too - will fix a few more before moving to review soon...

@ddobrigk ddobrigk marked this pull request as ready for review September 18, 2025 11:43
@ddobrigk
Copy link
Collaborator Author

Ready for review - this now also fixes a number of linter issues, such as preferring PDG_t versus hardcoded PDGs as well as TMCProcess::kPDecay. Maybe @romainschotter and @ercolessi: this isn't urgent to merge, so before approving, please go through the changes I did and inspect for typos, just in case. Thank you very much!

@romainschotter romainschotter enabled auto-merge (squash) September 18, 2025 16:38
@romainschotter romainschotter merged commit 077210d into AliceO2Group:master Sep 18, 2025
13 of 14 checks passed
jmunozme pushed a commit to jmunozme/O2Physics that referenced this pull request Oct 3, 2025
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Oct 11, 2025
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
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